On 2015-08-13 00:24:29 -0300, Alvaro Herrera wrote:
> Andres Freund wrote:
> > On 2015-08-12 16:08:08 -0300, Alvaro Herrera wrote:
> > > Alvaro Herrera wrote:
> > > > Close some holes in BRIN page assignment
> > >
> > > buildfarm evidently didn't like this one :-(
> >
> > clang seems to see a (the?) problem:
>
> Ahh, right. There's an identical problem in the other function
> (brin_doupdate); maybe the conditional there is more complicated than it
> wants to analyze. I was trying randomized memory and wasn't finding
> anything ...
>
> BTW I looked around the buildfarm and couldn't find a single member that
> displayed these warnings :-(
I tripped on clang 3.7 (unreleased) and a fairly extensive set of
warning flags (-Weverything + -Wno-... of the stupid ones)...
FWIW, this very likely would have tripped in valgrind. Just running that
single test will not even be too slow.
Regards,
Andres