dejagnu
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[DejaGnu] dejagnu patch for IEEE compile flags


From: Geoffrey Keating
Subject: [DejaGnu] dejagnu patch for IEEE compile flags
Date: Fri, 1 Jun 2001 15:39:05 -0700

The attached patch is to parallel a change I'll make to the GCC
testsuite.  It used to be that the IEEE FP tests of GCC used
-ffloat-store unconditionally.  On nearly all systems, this is
unnecessary.  The last straw was that we had a testcase, that depended
on IEEE FP, but which failed without -ffloat-store on powerpc and
passed with it.

This adds -ffloat-store to ieee_multilib_flags for x86, just as -mieee
is added for Alpha.  -ffloat-store is apparently also needed for m68k
platforms older than the 68040, but I don't think we have any
baseboards that are always like that, so it will need to be added in
site-specific files depending on the hardware on the actual board.

Is this OK to apply?

-- 
Geoff Keating <address@hidden>

===File ~/patches/dejagnu-floatstore.patch==================
2001-06-01  Geoffrey Keating  <address@hidden>

        * baseboards/unix.exp: Add -ffloat-store to ieee_multilib_flags
        on x86 targets.
        * baseboards/x86-cygmon.exp: Likewise.
        * baseboards/i386-bozo.exp: Likewise.

Index: src/dejagnu/baseboards/i386-bozo.exp
===================================================================
RCS file: /cvs/src/src/dejagnu/baseboards/i386-bozo.exp,v
retrieving revision 1.1.1.1
diff -p -u -p -r1.1.1.1 i386-bozo.exp
--- src/dejagnu/baseboards/i386-bozo.exp        1999/11/09 01:28:42     1.1.1.1
+++ src/dejagnu/baseboards/i386-bozo.exp        2001/06/01 22:15:13
@@ -49,3 +49,6 @@ set_board_info needs_status_wrapper 1
 
 # This is the start symbol in crt0.o.
 set_board_info gdb,start_symbol "_start";
+
+# x86 GCC needs -ffloat-store to be truly IEEE.  (This is probably a bug.)
+set_board_info ieee_multilib_flags "-ffloat-store";
Index: src/dejagnu/baseboards/unix.exp
===================================================================
RCS file: /cvs/src/src/dejagnu/baseboards/unix.exp,v
retrieving revision 1.2
diff -p -u -p -r1.2 unix.exp
--- src/dejagnu/baseboards/unix.exp     2001/04/04 23:20:34     1.2
+++ src/dejagnu/baseboards/unix.exp     2001/06/01 22:15:13
@@ -10,3 +10,8 @@ set_board_info bmk,use_alarm 1;
 if { [istarget "alpha*-*"] } {
   set_board_info ieee_multilib_flags "-mieee";
 }
+
+if { [istarget "i?86*-*"] } {
+  # x86 GCC needs -ffloat-store to be truly IEEE.  (This is probably a bug.)
+  set_board_info ieee_multilib_flags "-ffloat-store";
+}
Index: src/dejagnu/baseboards/x86-cygmon.exp
===================================================================
RCS file: /cvs/src/src/dejagnu/baseboards/x86-cygmon.exp,v
retrieving revision 1.1.1.1
diff -p -u -p -r1.1.1.1 x86-cygmon.exp
--- src/dejagnu/baseboards/x86-cygmon.exp       1999/11/09 01:28:42     1.1.1.1
+++ src/dejagnu/baseboards/x86-cygmon.exp       2001/06/01 22:15:13
@@ -9,3 +9,5 @@ if { [board_info $board obj_format] == "
     set_board_info gdb,start_symbol "__start"
 }
 
+# x86 GCC needs -ffloat-store to be truly IEEE.  (This is probably a bug.)
+set_board_info ieee_multilib_flags "-ffloat-store";
============================================================



reply via email to

[Prev in Thread] Current Thread [Next in Thread]