PATCH: Update default arch for x86

[H.J. Lu]

On Sat, Feb 27, 2010 at 6:22 PM, Jack Howarth <howarth@bromo.med.uc.edu> wrote:
> On Sat, Feb 27, 2010 at 01:36:07PM -0800, H.J. Lu wrote:
>>
>> This should work.
>>
>> --
>> H.J.
>
>> 2010-02-27  H.J. Lu  <hongjiu.lu@intel.com>
>>
>>       * config.gcc: Set the default 32bit/64bit archs with $with_arch
>>       instead of $arch for i[34567]86-*-*|x86_64-*-* targets.
>>
>> diff --git a/gcc/config.gcc b/gcc/config.gcc
>> index 0d46878..28e69ba 100644
>> --- a/gcc/config.gcc
>> +++ b/gcc/config.gcc
>> @@ -2886,7 +2886,7 @@ if test x$with_arch = x ; then
>>      i[34567]86-*-*|x86_64-*-*)
>>        if test x$with_arch_32 != x || test x$with_arch_64 != x; then
>>       if test x$with_arch_32 = x; then
>> -       with_arch_32=$arch
>> +       with_arch_32=$with_arch
>>       fi
>>       if test x$with_arch_64 = x; then
>>         if test $arch_without_64bit = yes; then
>> @@ -2894,14 +2894,14 @@ if test x$with_arch = x ; then
>>           # doesn't support 64bit.
>>           with_arch_64=x86-64
>>         else
>> -         with_arch_64=$arch
>> +         with_arch_64=$with_arch
>>         fi
>>       fi
>>       with_arch=
>>        elif test $arch_without_64bit$need_64bit_isa = yesyes; then
>>       # Set the default 64bit arch to x86-64 if the default arch
>>       # doesn't support 64bit and we need 64bit ISA.
>> -     with_arch_32=$arch
>> +     with_arch_32=$with_arch
>>       with_arch_64=x86-64
>>       with_arch=
>>        fi
>
>
> HJ,
>   I noticed that with this proposed patch and current gcc trunk, the default optimizations
> seem to be lower now for x86_64-apple-darwin10 than for i686-apple-darwin10...
>

I can make them the same by defaulting i686-apple-darwin to pentium4.

-- 
H.J.