Opened 11 years ago

Closed 3 years ago

#504 closed task (invalid)

Consider replacing -ffloat-store with -fexcess-precision=standard

Reported by: strk Owned by: geos-devel@…
Priority: major Milestone: GEOS Fund Me
Component: Default Version: 3.3.1
Severity: Unassigned Keywords:
Cc:

Description

See: http://archives.postgresql.org/pgsql-hackers/2011-12/msg00647.php

And see if such switch would bring us closer to JTS numeric handling

Change History (4)

comment:1 by strk, 11 years ago

g++ 4.4.3 doesn't seem to support the switch, but the AC_LIBTOOL_COMPILER_OPTION would think it does. AC_LIBTOOL_LINKER_OPTION instead belives it doesn't.

Could someone with a newer GCC try it there ?

Here's the patch:

diff --git a/configure.in b/configure.in
index 4ef0ef6..5f60996 100644
--- a/configure.in
+++ b/configure.in
@@ -155,6 +155,7 @@ AC_LIBTOOL_COMPILER_OPTION([if $compiler supports -Wno-long-long], [dummy_cv_no_
 # To make numerical computation more stable, we use --ffloat-store
 NUMERICFLAGS=""
 AC_LIBTOOL_COMPILER_OPTION([if $compiler supports -ffloat-store], [dummy_cv_ffloat_store], [-ffloat-store], [], [NUMERICFLAGS="$NUMERICFLAGS -ffloat-store"], [])
+AC_LIBTOOL_LINKER_OPTION([if $compiler supports -fexcess-precision=standard], [dummy_cv_excess_precision], [-excess-precision=standard], [], [NUMERICFLAGS="$NUMERICFLAGS -excess-precision=standard"], [])
 
 DEFAULTFLAGS="${WARNFLAGS} ${NUMERICFLAGS}"

comment:2 by strk, 11 years ago

Milestone: 3.3.2GEOS Future

comment:3 by robe, 4 years ago

Milestone: GEOS FutureGEOS Fund Me

Milestone renamed

comment:4 by pramsey, 3 years ago

Resolution: invalid
Status: newclosed

The world has moved on.

Note: See TracTickets for help on using tickets.