From: | Peter Eisentraut <peter_e(at)gmx(dot)net> |
---|---|
To: | pgsql-hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: new clang report |
Date: | 2011-05-02 17:54:47 |
Message-ID: | 1304358887.10895.7.camel@vanquo.pezone.net |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
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
> SELECT '' AS bad, f.f1 ^ '1e200' from FLOAT8_TBL f;
> -ERROR: value out of range: overflow
> + bad | ?column?
> +-----+----------
> + | 0
> + | NaN
> + | NaN
> + | NaN
> + | NaN
> +(5 rows)
> +
> SELECT 0 ^ 0 + 0 ^ 1 + 0 ^ 0.0 + 0 ^ 0.5;
> ?column?
> ----------
So issue here is actually that clang has an option
-fmath-errno
Indicate that math functions should be treated as updating errno.
If you pass this option, then the regression tests pass. If not, you
get the above difference. So the question is, do we
a) legislate that -fmath-errno is required, or
b) fix dpow() to handle this case somehow (how?), or
c) provide an alternative expected file?
From | Date | Subject | |
---|---|---|---|
Next Message | Josh Berkus | 2011-05-02 18:12:51 | A small step towards more organized beta testing |
Previous Message | David Fetter | 2011-05-02 17:30:15 | Re: HTML tags :/ |