[Top][All Lists]
[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";
============================================================
- [DejaGnu] dejagnu patch for IEEE compile flags,
Geoffrey Keating <=