This is something I had first done and tested a while ago, but
apparently failed to submit.

It did not show in my running of the testsuite (bootstrap and
testing without regressions on i386-unknown-freebsd10.1), but
in some cases the use of i586 instructions actually avoided
issues for users.  And who is running i486 these days?

(And FreeBSD 5 has been out of support for many years; I guess
modern versions of GCC won't work there for completely different
reasons.)

Okay?

And okay for GCC 5 after two weeks or so on trunk?

Gerald

2015-11-15  Gerald Pfeifer  <ger...@pfeifer.com>

        * config/i386/freebsd.h (SUBTARGET32_DEFAULT_CPU): Change to i586.
        Remove support for FreeBSD 5 and earlier.

Index: gcc/config/i386/freebsd.h
===================================================================
--- gcc/config/i386/freebsd.h   (revision 230394)
+++ gcc/config/i386/freebsd.h   (working copy)
@@ -121,10 +121,7 @@
 /* Static stack checking is supported by means of probes.  */
 #define STACK_CHECK_STATIC_BUILTIN 1
 
-/* Support for i386 has been removed from FreeBSD 6.0 onward.  */
-#if FBSD_MAJOR >= 6
-#define SUBTARGET32_DEFAULT_CPU "i486"
-#endif
+#define SUBTARGET32_DEFAULT_CPU "i586"
 
 #define TARGET_ASM_FILE_END file_end_indicate_exec_stack
 

Reply via email to