Patch applied. Thanks. ---------------------------------------------------------------------------
Jeremy Drake wrote: > On Sat, 22 Apr 2006, Tom Lane wrote: > > > Given that we've already got a test for ICC in there as of today, I'd > > vote for adding -mp1 to CFLAGS if we see it's ICC. > > This patch seems to do the trick... > > Index: configure.in > =================================================================== > RCS file: /home/jeremyd/local/postgres/cvsuproot/pgsql/configure.in,v > retrieving revision 1.460 > diff -c -r1.460 configure.in > *** configure.in 22 Apr 2006 00:29:41 -0000 1.460 > --- configure.in 25 Apr 2006 06:03:12 -0000 > *************** > *** 263,268 **** > --- 263,273 ---- > # Check whether they are supported, and add them to CFLAGS if so. > PGAC_PROG_CC_CFLAGS_OPT([-Wdeclaration-after-statement]) > PGAC_PROG_CC_CFLAGS_OPT([-Wendif-labels]) > + else > + # Intel compiler has a bug/misoptimization in checking for > + # division by NAN (NaN == 0), -mp1 fixes it, so add it to the > + # CFLAGS. > + PGAC_PROG_CC_CFLAGS_OPT([-mp1]) > fi > > # Disable strict-aliasing rules; needed for gcc 3.3+ > > > > > -- > A quarrel is quickly settled when deserted by one party; there is no battle > unless there be two. -- Seneca > > ---------------------------(end of broadcast)--------------------------- > TIP 9: In versions below 8.0, the planner will ignore your desire to > choose an index scan if your joining column's datatypes do not > match > -- Bruce Momjian http://candle.pha.pa.us EnterpriseDB http://www.enterprisedb.com + If your life is a hard drive, Christ can be your backup. + ---------------------------(end of broadcast)--------------------------- TIP 5: don't forget to increase your free space map settings