This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [patch,i386] Cannot inline sse*.* functions into avx functions


On Mon, Jul 1, 2013 at 2:12 PM, Uros Bizjak <ubizjak@gmail.com> wrote:
> On Mon, Jul 1, 2013 at 8:01 PM, Sriraman Tallam <tmsriram@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.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]