[geos-devel] Re: [GEOS] #504: Consider replacing -ffloat-store with -fexcess-precision=standard

[GEOS]

#504: Consider replacing -ffloat-store with  -fexcess-precision=standard
------------------------+---------------------------------------------------
 Reporter:  strk        |       Owner:  geos-devel@…              
     Type:  task        |      Status:  new                       
 Priority:  major       |   Milestone:  3.3.2                     
Component:  Default     |     Version:  3.3.1                     
 Severity:  Unassigned  |    Keywords:                            
------------------------+---------------------------------------------------

Comment(by strk):

 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}"
 }}}

-- 
Ticket URL: <http://trac.osgeo.org/geos/ticket/504#comment:1>
GEOS <http://geos.refractions.net/>
GEOS (Geometry Engine - Open Source) is a C++ port of the Java Topology Suite (JTS).