This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: PATCH: Update default arch for x86
On Sun, Feb 28, 2010 at 9:44 AM, Jack Howarth <howarth@bromo.med.uc.edu> wrote:
> On Sun, Feb 28, 2010 at 07:34:40AM -0800, H.J. Lu wrote:
>> On Sun, Feb 28, 2010 at 7:12 AM, Jack Howarth <howarth@bromo.med.uc.edu> wrote:
>> > On Sat, Feb 27, 2010 at 10:34:15PM -0800, H.J. Lu wrote:
>> >>
>> >> I can make them the same by defaulting i686-apple-darwin to pentium4.
>> >>
>> >> --
>> >> H.J.
>> >
>> > HJ,
>> > ? I was hoping to make x86_64-apple-darwin10 higher instead so that it
>> > supports -msse3 by default as was i686-apple-darwin10 in your previous
>> > patch. Note that in Snow Leopard, Apple's system gcc-4.2.1 compiler shows...
>>
>> You can try this.
>>
>> ---
>> diff --git a/gcc/config/i386/darwin.h b/gcc/config/i386/darwin.h
>> index eb9ee70..49863a6 100644
>> --- a/gcc/config/i386/darwin.h
>> +++ b/gcc/config/i386/darwin.h
>> @@ -162,14 +162,12 @@ extern void darwin_x86_file_end (void);
>> ?#undef TARGET_SUBTARGET32_ISA_DEFAULT
>> ?#define TARGET_SUBTARGET32_ISA_DEFAULT (OPTION_MASK_ISA_MMX ? ? ? ? ?\
>> ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? | OPTION_MASK_ISA_SSE ? ? ? ? ? \
>> - ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? | OPTION_MASK_ISA_SSE2)
>> -
>> -#undef TARGET_SUBTARGET64_ISA_DEFAULT
>> -#define TARGET_SUBTARGET64_ISA_DEFAULT (OPTION_MASK_ISA_MMX ? ? ? ? ?\
>> - ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? | OPTION_MASK_ISA_SSE ? ? ? ? ? \
>> ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? | OPTION_MASK_ISA_SSE2 ? ? ? ? ?\
>> ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? | OPTION_MASK_ISA_SSE3)
>>
>> +#undef TARGET_SUBTARGET64_ISA_DEFAULT
>> +#define TARGET_SUBTARGET64_ISA_DEFAULT TARGET_SUBTARGET32_ISA_DEFAULT
>> +
>> ?/* For now, disable dynamic-no-pic. ?We'll need to go through i386.c
>> ? ? with a fine-tooth comb looking for refs to flag_pic! ?*/
>> ?#define MASK_MACHO_DYNAMIC_NO_PIC 0
>> ----
>>
>>
>
> Can you check that change into gcc trunk?
> ? ? ? ? ? ? ? ? ? Jack
Done.
> ps Would it be a net win for darwin if we also set...
>
> with_cpu=${with_cpu:-core2}
>
> for both i[34567]86-*-darwin* and x86_64-*-darwin* in gcc/config.gcc
> like Apple's gcc 4.2.1 compiler to set -mtune= to core2?
>
-mtune=core2 was added by Apple. I found it was slower than
-mtune=generic on Linux/Core 2. I don't know how it performs
against -mtune=generic on Darwin.
--
H.J.