Patchwork Define SUBTARGET32_DEFAULT_CPU on FreeBSD

login
register
mail settings
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

Tijl Coosemans - Sept. 17, 2010, 12:59 p.m.
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.
Gerald Pfeifer - Sept. 17, 2010, 2:09 p.m.
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
Richard Henderson - Sept. 17, 2010, 8:38 p.m.
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~
Gerald Pfeifer - Sept. 18, 2010, 5:10 p.m.
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