On Mon, Jul 1, 2013 at 2:12 PM, Uros Bizjak <ubiz...@gmail.com> wrote:
> On Mon, Jul 1, 2013 at 8:01 PM, Sriraman Tallam <tmsri...@google.com> wrote:
>
>> So, something like the patch attached?
>>
>> * config/i386/i386.c (ix86_option_override_internal): Turn
>> on all -mavx target flags by default as they are dependent
>> on AVX anyway.
>
> Yes, but please also add:
>
> Index: i386.c
> ===================================================================
> --- i386.c (revision 200579)
> +++ i386.c (working copy)
> @@ -2578,7 +2578,7 @@ static const char *const cpu_names[TARGET_CPU_DEFA
> static bool
> gate_insert_vzeroupper (void)
> {
> - return TARGET_VZEROUPPER;
> + return TARGET_AVX && TARGET_VZEROUPPER;
> }
>
> static unsigned int
>
> so we won't trigger vzeroupper insertion pass for non-AVX targets.
Made that change, added the test case and committed.
Thanks
Sri
>
> Uros.
