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. --- 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