This is the mail archive of the gdb-patches@sources.redhat.com mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]

dejagnu patch for IEEE compile flags



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 <geoffk@redhat.com>

===File ~/patches/dejagnu-floatstore.patch==================
2001-06-01  Geoffrey Keating  <geoffk@redhat.com>

	* 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";
============================================================


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]