| Submitter | Tijl Coosemans |
|---|---|
| Date | Sept. 17, 2010, 12:59 p.m. |
| Message ID | <201009171459.56955.tijl@coosemans.org> |
| Download | mbox | patch |
| Permalink | /patch/65073/ |
| State | New |
| Headers | show |
Comments
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
Patch
--- 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.