On Wed, Feb 21, 2018 at 03:19:22PM +0100, Martin Liška wrote:
> On 02/21/2018 03:08 PM, Jakub Jelinek wrote:
> > This just adds "native" as possible value for -march, but shouldn't it be
> > also for -mtune, i.e. around line 4268?
>
> Thanks for note. There's updated version.
>
> Is it ok now?
> Thanks,
> Martin
> >From 980016be748b8f6a917f497d256c62a054bdece8 Mon Sep 17 00:00:00 2001
> From: marxin <mli...@suse.cz>
> Date: Tue, 20 Feb 2018 14:21:05 +0100
> Subject: [PATCH] Add "native" as a valid option value for -march= on i386 (PR
> driver/83193).
>
> gcc/ChangeLog:
>
> 2018-02-20 Martin Liska <mli...@suse.cz>
>
> PR driver/83193
> * config/i386/i386.c (ix86_option_override_internal):
> Add "native" as a possible value for -march and -mtune.
Ok, thanks.
> ---
> gcc/config/i386/i386.c | 10 ++++++++++
> 1 file changed, 10 insertions(+)
>
> diff --git a/gcc/config/i386/i386.c b/gcc/config/i386/i386.c
> index d54e7301e84..18d9084fd30 100644
> --- a/gcc/config/i386/i386.c
> +++ b/gcc/config/i386/i386.c
> @@ -4193,6 +4193,11 @@ ix86_option_override_internal (bool main_args_p,
> || ((processor_alias_table[i].flags & PTA_64BIT) != 0)))
> candidates.safe_push (processor_alias_table[i].name);
>
> +#ifdef HAVE_LOCAL_CPU_DETECT
> + /* Add also "native" as possible value. */
> + candidates.safe_push ("native");
> +#endif
> +
> char *s;
> const char *hint
> = candidates_list_and_hint (opts->x_ix86_arch_string, s, candidates);
> @@ -4265,6 +4270,11 @@ ix86_option_override_internal (bool main_args_p,
> || ((processor_alias_table[i].flags & PTA_64BIT) != 0))
> candidates.safe_push (processor_alias_table[i].name);
>
> +#ifdef HAVE_LOCAL_CPU_DETECT
> + /* Add also "native" as possible value. */
> + candidates.safe_push ("native");
> +#endif
> +
> char *s;
> const char *hint
> = candidates_list_and_hint (opts->x_ix86_tune_string, s, candidates);
> --
> 2.16.1
>
Jakub
