Thread: pgsql-server/. configure configure.in

pgsql-server/. configure configure.in

From
petere@postgresql.org (Peter Eisentraut - PostgreSQL)
Date:
CVSROOT:    /cvsroot
Module name:    pgsql-server
Changes by:    petere@postgresql.org    02/09/20 14:38:57

Modified files:
    .              : configure configure.in

Log message:
    Put in defense against gcc -ffast-math.


Re: pgsql-server/. configure configure.in

From
Tom Lane
Date:
petere@postgresql.org (Peter Eisentraut - PostgreSQL) writes:
> Log message:
>     Put in defense against gcc -ffast-math.

It occurs to me that it might also be a good idea to put something like
this in timestamp.c:

    #ifdef __FAST_MATH__
    #error -ffast-math is known to break this code
    #endif

to guard against people modifying the CFLAGS after running configure.
The configure test is good, because it catches the problem sooner,
but it's not foolproof.

Comments?

            regards, tom lane

Re: pgsql-server/. configure configure.in

From
Peter Eisentraut
Date:
Tom Lane writes:

> It occurs to me that it might also be a good idea to put something like
> this in timestamp.c:
>
>     #ifdef __FAST_MATH__
>     #error -ffast-math is known to break this code
>     #endif

Good idea.

--
Peter Eisentraut   peter_e@gmx.net