| Message ID | 201009171459.56955.tijl@coosemans.org |
|---|---|
| State | New |
| Headers | show |
I'll be happy to commit (and post test results) if anyone wants to aprove. :-) Gerald On Fri, 17 Sep 2010, Tijl Coosemans wrote: > Support for the i386 cpu has been removed from FreeBSD 6.0 onward and > thus i486 instructions can be used by default. A similar change has been > applied to the version of GCC supplied with FreeBSD: > http://svn.freebsd.org/viewvc/base?view=revision&revision=198344 > > I don't have commit access so feel free to commit this when approved. > > Tijl Coosemans > > * config/i386/freebsd.h (SUBTARGET32_DEFAULT_CPU): Add. > > --- config/i386/freebsd.h.orig > +++ config/i386/freebsd.h > @@ -141,3 +141,8 @@ > > /* 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
On 09/17/2010 07:09 AM, Gerald Pfeifer wrote: > I'll be happy to commit (and post test results) if anyone wants to > aprove. :-) > > Gerald > > On Fri, 17 Sep 2010, Tijl Coosemans wrote: >> Support for the i386 cpu has been removed from FreeBSD 6.0 onward and >> thus i486 instructions can be used by default. A similar change has been >> applied to the version of GCC supplied with FreeBSD: >> http://svn.freebsd.org/viewvc/base?view=revision&revision=198344 >> >> I don't have commit access so feel free to commit this when approved. >> >> Tijl Coosemans >> >> * config/i386/freebsd.h (SUBTARGET32_DEFAULT_CPU): Add. >> >> --- config/i386/freebsd.h.orig >> +++ config/i386/freebsd.h >> @@ -141,3 +141,8 @@ >> >> /* 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 The patch is ok. r~
On Fri, 17 Sep 2010, Richard Henderson wrote: >> I'll be happy to commit (and post test results) if anyone wants to >> aprove. :-) >>> * config/i386/freebsd.h (SUBTARGET32_DEFAULT_CPU): Add. > The patch is ok. Thanks, Richard. Applied as revision 164396, testresults at http://gcc.gnu.org/ml/gcc-testresults/2010-09/msg01637.html http://gcc.gnu.org/ml/gcc-testresults/2010-09/msg01663.html Gerald
--- config/i386/freebsd.h.orig +++ config/i386/freebsd.h @@ -141,3 +141,8 @@ /* 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
Support for the i386 cpu has been removed from FreeBSD 6.0 onward and thus i486 instructions can be used by default. A similar change has been applied to the version of GCC supplied with FreeBSD: http://svn.freebsd.org/viewvc/base?view=revision&revision=198344 I don't have commit access so feel free to commit this when approved. Tijl Coosemans * config/i386/freebsd.h (SUBTARGET32_DEFAULT_CPU): Add.