This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Add "native" as a valid option value for -march= on i386 (PR driver/83193).
- From: Richard Sandiford <richard dot sandiford at linaro dot org>
- To: Martin Liška <mliska at suse dot cz>
- Cc: gcc-patches at gcc dot gnu dot org, Jakub Jelinek <jakub at redhat dot com>, Jan Hubicka <hubicka at ucw dot cz>
- Date: Tue, 20 Feb 2018 15:58:07 +0000
- Subject: Re: [PATCH] Add "native" as a valid option value for -march= on i386 (PR driver/83193).
- Authentication-results: sourceware.org; auth=none
- References: <f6ef8fc4-43b4-8e1a-34ef-3379ea4d69c7@suse.cz>
Martin Liška <mliska@suse.cz> writes:
> Hi.
>
> Following patch adds "native" as a possible option for -march value on
> i386 target. I have similar patches for other targets. Would it be
> possible to install the patch in current stage?
[...]
> diff --git a/gcc/config/i386/i386.c b/gcc/config/i386/i386.c
> index d54e7301e84..361d4df2663 100644
> --- a/gcc/config/i386/i386.c
> +++ b/gcc/config/i386/i386.c
> @@ -4193,6 +4193,9 @@ ix86_option_override_internal (bool main_args_p,
> || ((processor_alias_table[i].flags & PTA_64BIT) != 0)))
> candidates.safe_push (processor_alias_table[i].name);
>
> + /* Add also "native" as possible value. */
> + candidates.safe_push ("native");
> +
> char *s;
> const char *hint
> = candidates_list_and_hint (opts->x_ix86_arch_string, s, candidates);
We should probably only do this if the driver supports -march=native.
I think for x86 that means HAVE_LOCAL_CPU_DETECT.
Thanks,
Richard