| From: | "Magnus Hagander" <mha(at)sollentuna(dot)net> | 
|---|---|
| To: | "Tom Lane" <tgl(at)sss(dot)pgh(dot)pa(dot)us> | 
| Cc: | <pgsql-hackers(at)postgresql(dot)org> | 
| Subject: | Re: Regression error on float8 | 
| Date: | 2006-04-24 13:01:51 | 
| Message-ID: | 6BCB9D8A16AC4241919521715F4D8BCEA352EC@algol.sollentuna.se | 
| Views: | Whole Thread | Raw Message | Download mbox | Resend email | 
| Thread: | |
| Lists: | pgsql-hackers | 
> > I'm getting the following regression errors with a backend 
> built using 
> > Visual C++:
> 
> Is HAVE_CBRT getting defined?  Either their cbrt() routine or 
> our default one seems to be generating slightly-off answers.  
> The default one (at the bottom of float.c) certainly looks a 
> bit cheesy, but if it fails this test you'd think we'd have 
> heard about that sooner.
HAVE_CBRT is not set.
If I undefine HAVE_CBRT on Linux, I get the exact same failure! So it
seems our own version of cbrt() is broken wrt our own regression tests
:-( Must be that nobody else (at least on i386) uses that code.
The mingw version does appear to work, but it's noticably more complex,
see
http://cvs.sourceforge.net/viewcvs.py/mingw/runtime/mingwex/math/cbrt.c?
rev=1.1&view=auto.
It's placed in the public domain, so we should be able to use it if we
want to
(http://cvs.sourceforge.net/viewcvs.py/mingw/runtime/DISCLAIMER?rev=1.1&
view=auto).
What do you think is best - try to adapt that version, or update our
regression tests outputs to accept the output from our current code?
//Magnus
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Alvaro Herrera | 2006-04-24 13:02:07 | Re: TODO Item: ACL_CONNECT | 
| Previous Message | Gevik Babakhani | 2006-04-24 12:54:25 | 2x compile warning |