Re: [HACKERS] new clang report

2011-05-05 Thread Tom Lane
Peter Eisentraut writes: > Btw., when you build a simple test program in the default mode, pow() > indeed returns Inf on overflow. There appear to be some code generation > or optimization problems when it builds the postgres code, because the > problem goes away with either -O0 or by inserting a

Re: [HACKERS] new clang report

2011-05-05 Thread Peter Eisentraut
On mån, 2011-05-02 at 14:45 -0400, Tom Lane wrote: > > So issue here is actually that clang has an option > > >-fmath-errno > >Indicate that math functions should be treated as > updating errno. > > Really? It looks to me like the issue is that pow() is returning NaN > instea

Re: [HACKERS] new clang report

2011-05-02 Thread Tom Lane
Peter Eisentraut writes: > On ons, 2011-02-09 at 20:30 +0200, Peter Eisentraut wrote: >> Regression tests (world): >> >> --- src/test/regress/expected/float8.out >> +++ src/test/regress/results/float8.out >> @@ -384,7 +384,15 @@ >> SELECT '' AS bad, f.f1 * '1e200' from FLOAT8_TBL f; >> ERROR: va

Re: [HACKERS] new clang report

2011-05-02 Thread Peter Eisentraut
On ons, 2011-02-09 at 20:30 +0200, Peter Eisentraut wrote: > Regression tests (world): > > --- src/test/regress/expected/float8.out > +++ src/test/regress/results/float8.out > @@ -384,7 +384,15 @@ > SELECT '' AS bad, f.f1 * '1e200' from FLOAT8_TBL f; > ERROR: value out of range: overflow > SEL

Re: [HACKERS] new clang report

2011-02-16 Thread Bruce Momjian
Peter Eisentraut wrote: > The lastest clang svn tip (2.9-to-be, I guess) builds PostgreSQL out of > the box and most tests pass. Specifically, it no longer chokes on > -D_GNU_SOURCE on Linux, which was the previously reported blocker. > > Warnings: > > Lots of these: > clang: warning: argument u

Re: [HACKERS] new clang report

2011-02-09 Thread Greg Stark
On Wed, Feb 9, 2011 at 6:30 PM, Peter Eisentraut wrote: > The lastest clang svn tip (2.9-to-be, I guess) builds PostgreSQL out of > the box and most tests pass.  Specifically, it no longer chokes on > -D_GNU_SOURCE on Linux, which was the previously reported blocker. Odd, I tried the same thing j

[HACKERS] new clang report

2011-02-09 Thread Peter Eisentraut
The lastest clang svn tip (2.9-to-be, I guess) builds PostgreSQL out of the box and most tests pass. Specifically, it no longer chokes on -D_GNU_SOURCE on Linux, which was the previously reported blocker. Warnings: Lots of these: clang: warning: argument unused during compilation: '-mthreads' cl