On Fri, Nov 29, 2013 at 7:39 PM, Greg Stark <st...@mit.edu> wrote: > > Just as an update I did get gcc to do the wrong thing on purpose. The > only overflow check that the regression tests find missing is the one > for int8abs() ie:
Also, one of the places GCC warns about optimizing away an overflow check (with -fno-wrapv) is inside the localtime.c file from the tz library. I fixed it in my patch but in fact I checked and it's already fixed upstream so I'm wondering whether you expect to merge in an updated tz library? Is there anything surprising about the process or do you just copy in the files? Would you be happy for someone else to do it? -- greg -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
