* [PATCH] Add "native" as a valid option value for -march= on i386 (PR driver/83193). @ 2018-02-20 13:25 Martin Liška 2018-02-20 15:58 ` Richard Sandiford ` (2 more replies) 0 siblings, 3 replies; 21+ messages in thread From: Martin Liška @ 2018-02-20 13:25 UTC (permalink / raw) To: gcc-patches; +Cc: Jakub Jelinek, Jan Hubicka [-- Attachment #1: Type: text/plain, Size: 2252 bytes --] 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? Before: $ ./xgcc -B. -march=abcdef /tmp/empty.c cc1: error: bad value (âabcdefâ) for â-march=â switch cc1: note: valid arguments to â-march=â switch are: nocona core2 nehalem corei7 westmere sandybridge corei7-avx ivybridge core-avx-i haswell core-avx2 broadwell skylake skylake-avx512 cannonlake icelake bonnell atom silvermont slm knl knm x86-64 eden-x2 nano nano-1000 nano-2000 nano-3000 nano-x2 eden-x4 nano-x4 k8 k8-sse3 opteron opteron-sse3 athlon64 athlon64-sse3 athlon-fx amdfam10 barcelona bdver1 bdver2 bdver3 bdver4 znver1 btver1 btver2 After: $ ./xgcc -B. -march=abcdef /tmp/empty.c cc1: error: bad value (âabcdefâ) for â-march=â switch cc1: note: valid arguments to â-march=â switch are: nocona core2 nehalem corei7 westmere sandybridge corei7-avx ivybridge core-avx-i haswell core-avx2 broadwell skylake skylake-avx512 cannonlake icelake bonnell atom silvermont slm knl knm x86-64 eden-x2 nano nano-1000 nano-2000 nano-3000 nano-x2 eden-x4 nano-x4 k8 k8-sse3 opteron opteron-sse3 athlon64 athlon64-sse3 athlon-fx amdfam10 barcelona bdver1 bdver2 bdver3 bdver4 znver1 btver1 btver2 native $ ./xgcc -B. -march=native2 /tmp/empty.c cc1: error: bad value (ânative2â) for â-march=â switch cc1: note: valid arguments to â-march=â switch are: nocona core2 nehalem corei7 westmere sandybridge corei7-avx ivybridge core-avx-i haswell core-avx2 broadwell skylake skylake-avx512 cannonlake icelake bonnell atom silvermont slm knl knm x86-64 eden-x2 nano nano-1000 nano-2000 nano-3000 nano-x2 eden-x4 nano-x4 k8 k8-sse3 opteron opteron-sse3 athlon64 athlon64-sse3 athlon-fx amdfam10 barcelona bdver1 bdver2 bdver3 bdver4 znver1 btver1 btver2 native; did you mean ânativeâ? Ready for trunk after tests? Thanks Martin gcc/ChangeLog: 2018-02-20 Martin Liska <mliska@suse.cz> PR driver/83193 * config/i386/i386.c (ix86_option_override_internal): Add "native" as a possible value. --- gcc/config/i386/i386.c | 3 +++ 1 file changed, 3 insertions(+) [-- Attachment #2: 0001-Add-native-as-a-valid-option-value-for-march-on-i386.patch --] [-- Type: text/x-patch, Size: 554 bytes --] 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); ^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH] Add "native" as a valid option value for -march= on i386 (PR driver/83193). 2018-02-20 13:25 [PATCH] Add "native" as a valid option value for -march= on i386 (PR driver/83193) Martin Liška @ 2018-02-20 15:58 ` Richard Sandiford 2018-02-20 17:49 ` Jakub Jelinek 2018-02-21 7:34 ` Add "native" as a valid option value for -march= on arm " Martin Liška 2018-02-21 7:35 ` [PATCH] Add "native" as a valid option value for -march= on aarch64 " Martin Liška 2 siblings, 1 reply; 21+ messages in thread From: Richard Sandiford @ 2018-02-20 15:58 UTC (permalink / raw) To: Martin Liška; +Cc: gcc-patches, Jakub Jelinek, Jan Hubicka 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 ^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH] Add "native" as a valid option value for -march= on i386 (PR driver/83193). 2018-02-20 15:58 ` Richard Sandiford @ 2018-02-20 17:49 ` Jakub Jelinek 2018-02-20 16:19 ` Richard Sandiford 0 siblings, 1 reply; 21+ messages in thread From: Jakub Jelinek @ 2018-02-20 17:49 UTC (permalink / raw) To: Martin Liška, gcc-patches, Jan Hubicka, richard.sandiford On Tue, Feb 20, 2018 at 03:58:07PM +0000, Richard Sandiford wrote: > 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. Isn't the option available always? It just doesn't do anything if /* If we are compiling with GCC where %EBX register is fixed, then the driver will just ignore -march and -mtune "native" target and will leave to the newly built compiler to generate code for its default target. */ Jakub ^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH] Add "native" as a valid option value for -march= on i386 (PR driver/83193). 2018-02-20 17:49 ` Jakub Jelinek @ 2018-02-20 16:19 ` Richard Sandiford 2018-02-20 16:26 ` Martin Liška 0 siblings, 1 reply; 21+ messages in thread From: Richard Sandiford @ 2018-02-20 16:19 UTC (permalink / raw) To: Jakub Jelinek; +Cc: Martin Liška, gcc-patches, Jan Hubicka Jakub Jelinek <jakub@redhat.com> writes: > On Tue, Feb 20, 2018 at 03:58:07PM +0000, Richard Sandiford wrote: >> 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. > > Isn't the option available always? It just doesn't do anything if > /* If we are compiling with GCC where %EBX register is fixed, then the > driver will just ignore -march and -mtune "native" target and will leave > to the newly built compiler to generate code for its default target. */ It's only available for x86 hosts: /* -march=native handling only makes sense with compiler running on an x86 or x86_64 chip. If changing this condition, also change the condition in driver-i386.c. */ #if defined(__i386__) || defined(__x86_64__) /* In driver-i386.c. */ extern const char *host_detect_local_cpu (int argc, const char **argv); #define EXTRA_SPEC_FUNCTIONS \ { "local_cpu_detect", host_detect_local_cpu }, #define HAVE_LOCAL_CPU_DETECT #endif Non-native hosts are obviously a niche case for x86, but it still seems better to be consistent. Richard ^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH] Add "native" as a valid option value for -march= on i386 (PR driver/83193). 2018-02-20 16:19 ` Richard Sandiford @ 2018-02-20 16:26 ` Martin Liška 2018-02-21 7:33 ` Martin Liška 0 siblings, 1 reply; 21+ messages in thread From: Martin Liška @ 2018-02-20 16:26 UTC (permalink / raw) To: Jakub Jelinek, gcc-patches, Jan Hubicka, richard.sandiford On 02/20/2018 05:19 PM, Richard Sandiford wrote: > Jakub Jelinek <jakub@redhat.com> writes: >> On Tue, Feb 20, 2018 at 03:58:07PM +0000, Richard Sandiford wrote: >>> 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. >> >> Isn't the option available always? It just doesn't do anything if >> /* If we are compiling with GCC where %EBX register is fixed, then the >> driver will just ignore -march and -mtune "native" target and will leave >> to the newly built compiler to generate code for its default target. */ > > It's only available for x86 hosts: > > /* -march=native handling only makes sense with compiler running on > an x86 or x86_64 chip. If changing this condition, also change > the condition in driver-i386.c. */ > #if defined(__i386__) || defined(__x86_64__) > /* In driver-i386.c. */ > extern const char *host_detect_local_cpu (int argc, const char **argv); > #define EXTRA_SPEC_FUNCTIONS \ > { "local_cpu_detect", host_detect_local_cpu }, > #define HAVE_LOCAL_CPU_DETECT > #endif > > Non-native hosts are obviously a niche case for x86, but it still > seems better to be consistent. > > Richard > So would it enough to wrap 'candidates.safe_push ("native");' by #ifdef HAVE_LOCAL_CPU_DETECT ? Thanks, Martin ^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH] Add "native" as a valid option value for -march= on i386 (PR driver/83193). 2018-02-20 16:26 ` Martin Liška @ 2018-02-21 7:33 ` Martin Liška 2018-02-21 19:49 ` Jakub Jelinek 0 siblings, 1 reply; 21+ messages in thread From: Martin Liška @ 2018-02-21 7:33 UTC (permalink / raw) To: Jakub Jelinek, gcc-patches, Jan Hubicka, richard.sandiford [-- Attachment #1: Type: text/plain, Size: 2407 bytes --] On 02/20/2018 05:26 PM, Martin LiÅ¡ka wrote: > On 02/20/2018 05:19 PM, Richard Sandiford wrote: >> Jakub Jelinek <jakub@redhat.com> writes: >>> On Tue, Feb 20, 2018 at 03:58:07PM +0000, Richard Sandiford wrote: >>>> 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. >>> >>> Isn't the option available always? It just doesn't do anything if >>> /* If we are compiling with GCC where %EBX register is fixed, then the >>> driver will just ignore -march and -mtune "native" target and will leave >>> to the newly built compiler to generate code for its default target. */ >> >> It's only available for x86 hosts: >> >> /* -march=native handling only makes sense with compiler running on >> an x86 or x86_64 chip. If changing this condition, also change >> the condition in driver-i386.c. */ >> #if defined(__i386__) || defined(__x86_64__) >> /* In driver-i386.c. */ >> extern const char *host_detect_local_cpu (int argc, const char **argv); >> #define EXTRA_SPEC_FUNCTIONS \ >> { "local_cpu_detect", host_detect_local_cpu }, >> #define HAVE_LOCAL_CPU_DETECT >> #endif >> >> Non-native hosts are obviously a niche case for x86, but it still >> seems better to be consistent. >> >> Richard >> > > So would it enough to wrap 'candidates.safe_push ("native");' by > #ifdef HAVE_LOCAL_CPU_DETECT > ? > > Thanks, > Martin > There's updated version of the patch I've just tested. Martin [-- Attachment #2: 0003-Add-native-as-a-valid-option-value-for-march-on-i386.patch --] [-- Type: text/x-patch, Size: 1081 bytes --] From 8f1783a9017ec06c578fd644e46168ec5763d5ca Mon Sep 17 00:00:00 2001 From: marxin <mliska@suse.cz> Date: Tue, 20 Feb 2018 14:21:05 +0100 Subject: [PATCH 3/3] Add "native" as a valid option value for -march= on i386 (PR driver/83193). gcc/ChangeLog: 2018-02-20 Martin Liska <mliska@suse.cz> PR driver/83193 * config/i386/i386.c (ix86_option_override_internal): Add "native" as a possible value. --- gcc/config/i386/i386.c | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/gcc/config/i386/i386.c b/gcc/config/i386/i386.c index d54e7301e84..9f2c5218ae5 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); -- 2.16.1 ^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH] Add "native" as a valid option value for -march= on i386 (PR driver/83193). 2018-02-21 7:33 ` Martin Liška @ 2018-02-21 19:49 ` Jakub Jelinek 2018-02-21 14:19 ` Martin Liška 0 siblings, 1 reply; 21+ messages in thread From: Jakub Jelinek @ 2018-02-21 19:49 UTC (permalink / raw) To: Martin Liška; +Cc: gcc-patches, Jan Hubicka, richard.sandiford On Wed, Feb 21, 2018 at 08:33:21AM +0100, Martin LiÅ¡ka wrote: > >From 8f1783a9017ec06c578fd644e46168ec5763d5ca Mon Sep 17 00:00:00 2001 > From: marxin <mliska@suse.cz> > Date: Tue, 20 Feb 2018 14:21:05 +0100 > Subject: [PATCH 3/3] Add "native" as a valid option value for -march= on i386 > (PR driver/83193). > > gcc/ChangeLog: > > 2018-02-20 Martin Liska <mliska@suse.cz> > > PR driver/83193 > * config/i386/i386.c (ix86_option_override_internal): > Add "native" as a possible value. > --- > gcc/config/i386/i386.c | 5 +++++ > 1 file changed, 5 insertions(+) > > diff --git a/gcc/config/i386/i386.c b/gcc/config/i386/i386.c > index d54e7301e84..9f2c5218ae5 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 This just adds "native" as possible value for -march, but shouldn't it be also for -mtune, i.e. around line 4268? > + > char *s; > const char *hint > = candidates_list_and_hint (opts->x_ix86_arch_string, s, candidates); > -- > 2.16.1 > Jakub ^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH] Add "native" as a valid option value for -march= on i386 (PR driver/83193). 2018-02-21 19:49 ` Jakub Jelinek @ 2018-02-21 14:19 ` Martin Liška 2018-02-21 14:23 ` Jakub Jelinek 0 siblings, 1 reply; 21+ messages in thread From: Martin Liška @ 2018-02-21 14:19 UTC (permalink / raw) To: Jakub Jelinek; +Cc: gcc-patches, Jan Hubicka, richard.sandiford [-- Attachment #1: Type: text/plain, Size: 236 bytes --] 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 [-- Attachment #2: 0001-Add-native-as-a-valid-option-value-for-march-on-i386.patch --] [-- Type: text/x-patch, Size: 1543 bytes --] From 980016be748b8f6a917f497d256c62a054bdece8 Mon Sep 17 00:00:00 2001 From: marxin <mliska@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 <mliska@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 ^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH] Add "native" as a valid option value for -march= on i386 (PR driver/83193). 2018-02-21 14:19 ` Martin Liška @ 2018-02-21 14:23 ` Jakub Jelinek 0 siblings, 0 replies; 21+ messages in thread From: Jakub Jelinek @ 2018-02-21 14:23 UTC (permalink / raw) To: Martin Liška; +Cc: gcc-patches, Jan Hubicka, richard.sandiford 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 <mliska@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 <mliska@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 ^ permalink raw reply [flat|nested] 21+ messages in thread
* Add "native" as a valid option value for -march= on arm (PR driver/83193). 2018-02-20 13:25 [PATCH] Add "native" as a valid option value for -march= on i386 (PR driver/83193) Martin Liška 2018-02-20 15:58 ` Richard Sandiford @ 2018-02-21 7:34 ` Martin Liška 2018-02-21 9:24 ` Kyrill Tkachov 2018-02-21 7:35 ` [PATCH] Add "native" as a valid option value for -march= on aarch64 " Martin Liška 2 siblings, 1 reply; 21+ messages in thread From: Martin Liška @ 2018-02-21 7:34 UTC (permalink / raw) To: gcc-patches Cc: Jakub Jelinek, Jan Hubicka, James Greenhalgh, Ramana Radhakrishnan, Kyrill Tkachov [-- Attachment #1: Type: text/plain, Size: 79 bytes --] Hi. This is equivalent patch for ARM target. Ready for trunk? Thanks, Martin [-- Attachment #2: 0001-Add-native-as-a-valid-option-value-for-march-on-arm-.patch --] [-- Type: text/x-patch, Size: 1810 bytes --] From 656a883bc5239439ba80743f15a8df704501ee71 Mon Sep 17 00:00:00 2001 From: marxin <mliska@suse.cz> Date: Tue, 20 Feb 2018 14:09:22 +0100 Subject: [PATCH 1/3] Add "native" as a valid option value for -march= on arm (PR driver/83193). gcc/ChangeLog: 2018-02-20 Martin Liska <mliska@suse.cz> PR driver/83193 * common/config/arm/arm-common.c (arm_print_hint_for_arch_option): Add "native" as a possible value. * config/arm/arm.h (HAVE_LOCAL_CPU_DETECT): Define the macro when native cpu detection is available. --- gcc/common/config/arm/arm-common.c | 6 ++++++ gcc/config/arm/arm.h | 1 + 2 files changed, 7 insertions(+) diff --git a/gcc/common/config/arm/arm-common.c b/gcc/common/config/arm/arm-common.c index fc585e0b0ee..50f0bad3e36 100644 --- a/gcc/common/config/arm/arm-common.c +++ b/gcc/common/config/arm/arm-common.c @@ -353,6 +353,12 @@ arm_print_hint_for_arch_option (const char *target, auto_vec<const char*> candidates; for (; list->common.name != NULL; list++) candidates.safe_push (list->common.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 (target, s, candidates); if (hint) diff --git a/gcc/config/arm/arm.h b/gcc/config/arm/arm.h index 6f3c4f461b9..bbf3937a592 100644 --- a/gcc/config/arm/arm.h +++ b/gcc/config/arm/arm.h @@ -2194,6 +2194,7 @@ extern const char *arm_target_thumb_only (int argc, const char **argv); an ARM chip. */ #if defined(__arm__) extern const char *host_detect_local_cpu (int argc, const char **argv); +#define HAVE_LOCAL_CPU_DETECT # define MCPU_MTUNE_NATIVE_FUNCTIONS \ { "local_cpu_detect", host_detect_local_cpu }, # define MCPU_MTUNE_NATIVE_SPECS \ -- 2.16.1 ^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: Add "native" as a valid option value for -march= on arm (PR driver/83193). 2018-02-21 7:34 ` Add "native" as a valid option value for -march= on arm " Martin Liška @ 2018-02-21 9:24 ` Kyrill Tkachov 2018-02-21 9:53 ` Martin Liška ` (2 more replies) 0 siblings, 3 replies; 21+ messages in thread From: Kyrill Tkachov @ 2018-02-21 9:24 UTC (permalink / raw) To: Martin Liška, gcc-patches Cc: Jakub Jelinek, Jan Hubicka, James Greenhalgh, Ramana Radhakrishnan Hi Martin, On 21/02/18 07:34, Martin LiÅ¡ka wrote: > Hi. > > This is equivalent patch for ARM target. Thanks for fixing this! > Ready for trunk? Have you tested this with a native and a cross-compiler like the aarch64 version? > Thanks, > Martin From 656a883bc5239439ba80743f15a8df704501ee71 Mon Sep 17 00:00:00 2001 From: marxin<mliska@suse.cz> Date: Tue, 20 Feb 2018 14:09:22 +0100 Subject: [PATCH 1/3] Add "native" as a valid option value for -march= on arm (PR driver/83193). gcc/ChangeLog: 2018-02-20 Martin Liska<mliska@suse.cz> PR driver/83193 * common/config/arm/arm-common.c (arm_print_hint_for_arch_option): Add "native" as a possible value. * config/arm/arm.h (HAVE_LOCAL_CPU_DETECT): Define the macro when native cpu detection is available. --- gcc/common/config/arm/arm-common.c | 6 ++++++ gcc/config/arm/arm.h | 1 + 2 files changed, 7 insertions(+) diff --git a/gcc/common/config/arm/arm-common.c b/gcc/common/config/arm/arm-common.c index fc585e0b0ee..50f0bad3e36 100644 --- a/gcc/common/config/arm/arm-common.c +++ b/gcc/common/config/arm/arm-common.c @@ -353,6 +353,12 @@ arm_print_hint_for_arch_option (const char *target, auto_vec<const char*> candidates; for (; list->common.name != NULL; list++) candidates.safe_push (list->common.name); + +#ifdef HAVE_LOCAL_CPU_DETECT + /* Add also "native" as possible value. */ + candidates.safe_push ("native"); +#endif + On arm we also support "native" as a value for -mcpu and -mtune. These are both handled by the arm_print_hint_for_cpu_option function in the same file. Can you please add this snippet to them as well? Thanks, Kyrill ^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: Add "native" as a valid option value for -march= on arm (PR driver/83193). 2018-02-21 9:24 ` Kyrill Tkachov @ 2018-02-21 9:53 ` Martin Liška 2018-02-21 12:26 ` Kyrill Tkachov 2018-02-21 14:03 ` Martin Liška 2018-02-22 8:52 ` Martin Liška 2 siblings, 1 reply; 21+ messages in thread From: Martin Liška @ 2018-02-21 9:53 UTC (permalink / raw) To: Kyrill Tkachov, gcc-patches Cc: Jakub Jelinek, Jan Hubicka, James Greenhalgh, Ramana Radhakrishnan On 02/21/2018 10:23 AM, Kyrill Tkachov wrote: > Have you tested this with a native and a cross-compiler like the aarch64 version? No, I don't have a machine. Can you please do that? Thanks, Martin ^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: Add "native" as a valid option value for -march= on arm (PR driver/83193). 2018-02-21 9:53 ` Martin Liška @ 2018-02-21 12:26 ` Kyrill Tkachov 0 siblings, 0 replies; 21+ messages in thread From: Kyrill Tkachov @ 2018-02-21 12:26 UTC (permalink / raw) To: Martin Liška, gcc-patches Cc: Jakub Jelinek, Jan Hubicka, James Greenhalgh, Ramana Radhakrishnan On 21/02/18 09:53, Martin LiÅ¡ka wrote: > On 02/21/2018 10:23 AM, Kyrill Tkachov wrote: >> Have you tested this with a native and a cross-compiler like the aarch64 version? > No, I don't have a machine. Can you please do that? Sure, I've bootstrapped and tested it on arm-none-linux-gnueabihf and an arm-none-eabi cross-compiler. Confirmed that "native" comes up as a suggestion for a native compiler and not for a cross-compiler. So the approach looks good. Please address the comment in my previous reply and this will be ok for trunk. Thanks, Kyrill > Thanks, > Martin ^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: Add "native" as a valid option value for -march= on arm (PR driver/83193). 2018-02-21 9:24 ` Kyrill Tkachov 2018-02-21 9:53 ` Martin Liška @ 2018-02-21 14:03 ` Martin Liška 2018-02-22 8:52 ` Martin Liška 2 siblings, 0 replies; 21+ messages in thread From: Martin Liška @ 2018-02-21 14:03 UTC (permalink / raw) To: Kyrill Tkachov, gcc-patches Cc: Jakub Jelinek, Jan Hubicka, James Greenhalgh, Ramana Radhakrishnan On 02/21/2018 10:23 AM, Kyrill Tkachov wrote: > Have you tested this with a native and a cross-compiler like the aarch64 version? Yes, you tested that ;) I'm going to install the patch. Martin ^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: Add "native" as a valid option value for -march= on arm (PR driver/83193). 2018-02-21 9:24 ` Kyrill Tkachov 2018-02-21 9:53 ` Martin Liška 2018-02-21 14:03 ` Martin Liška @ 2018-02-22 8:52 ` Martin Liška 2018-02-22 10:59 ` Kyrill Tkachov 2 siblings, 1 reply; 21+ messages in thread From: Martin Liška @ 2018-02-22 8:52 UTC (permalink / raw) To: Kyrill Tkachov, gcc-patches Cc: Jakub Jelinek, Jan Hubicka, James Greenhalgh, Ramana Radhakrishnan [-- Attachment #1: Type: text/plain, Size: 426 bytes --] On 02/21/2018 10:23 AM, Kyrill Tkachov wrote: > > On arm we also support "native" as a value for -mcpu and -mtune. These are both handled by > the arm_print_hint_for_cpu_option function in the same file. Can you please add this snippet > to them as well? Hi. Can you please test for me attached follow up patch? Thanks, Martin [-- Attachment #2: 0001-Add-native-as-a-valid-option-value-for-mcpu-mtune-on.patch --] [-- Type: text/x-patch, Size: 1143 bytes --] From 3a40e8a873461f9f06a18539cfb7c366dd6efafb Mon Sep 17 00:00:00 2001 From: marxin <mliska@suse.cz> Date: Wed, 21 Feb 2018 15:38:32 +0100 Subject: [PATCH] Add "native" as a valid option value for -mcpu/-mtune= on arm (PR driver/83193). gcc/ChangeLog: 2018-02-21 Martin Liska <mliska@suse.cz> PR driver/83193 * common/config/arm/arm-common.c (arm_print_hint_for_cpu_option): Add "native" as a possible value. --- gcc/common/config/arm/arm-common.c | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/gcc/common/config/arm/arm-common.c b/gcc/common/config/arm/arm-common.c index 50f0bad3e36..a404d4b1562 100644 --- a/gcc/common/config/arm/arm-common.c +++ b/gcc/common/config/arm/arm-common.c @@ -309,6 +309,12 @@ arm_print_hint_for_cpu_option (const char *target, auto_vec<const char*> candidates; for (; list->common.name != NULL; list++) candidates.safe_push (list->common.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 (target, s, candidates); if (hint) -- 2.16.1 ^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: Add "native" as a valid option value for -march= on arm (PR driver/83193). 2018-02-22 8:52 ` Martin Liška @ 2018-02-22 10:59 ` Kyrill Tkachov 2018-02-22 12:16 ` Martin Liška 0 siblings, 1 reply; 21+ messages in thread From: Kyrill Tkachov @ 2018-02-22 10:59 UTC (permalink / raw) To: Martin Liška, gcc-patches Cc: Jakub Jelinek, Jan Hubicka, James Greenhalgh, Ramana Radhakrishnan Hi Martin, On 22/02/18 08:51, Martin LiÅ¡ka wrote: > On 02/21/2018 10:23 AM, Kyrill Tkachov wrote: >> On arm we also support "native" as a value for -mcpu and -mtune. These are both handled by >> the arm_print_hint_for_cpu_option function in the same file. Can you please add this snippet >> to them as well? > Hi. > > Can you please test for me attached follow up patch? I've bootstrapped and tested this patch on a native and a cross compiler and confirmed the help string for -mcpu behaves as intended. This patch is ok for trunk. Thanks, Kyrill > Thanks, > Martin ^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: Add "native" as a valid option value for -march= on arm (PR driver/83193). 2018-02-22 10:59 ` Kyrill Tkachov @ 2018-02-22 12:16 ` Martin Liška 2018-03-08 9:02 ` Martin Liška 2018-03-09 15:43 ` Kyrill Tkachov 0 siblings, 2 replies; 21+ messages in thread From: Martin Liška @ 2018-02-22 12:16 UTC (permalink / raw) To: Kyrill Tkachov, gcc-patches Cc: Jakub Jelinek, Jan Hubicka, James Greenhalgh, Ramana Radhakrishnan On 02/22/2018 11:59 AM, Kyrill Tkachov wrote: > Hi Martin, > > On 22/02/18 08:51, Martin LiÅ¡ka wrote: >> On 02/21/2018 10:23 AM, Kyrill Tkachov wrote: >>> On arm we also support "native" as a value for -mcpu and -mtune. These are both handled by >>> the arm_print_hint_for_cpu_option function in the same file. Can you please add this snippet >>> to them as well? >> Hi. >> >> Can you please test for me attached follow up patch? > > I've bootstrapped and tested this patch on a native and a cross compiler and confirmed > the help string for -mcpu behaves as intended. > > This patch is ok for trunk. > Thanks, > Kyrill Thanks for review. May I please ask you for taking look at: 2b) point in: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83193#c6 We should fix it (if possible) for GCC 8 as it's regression. Thanks, Martin > >> Thanks, >> Martin > ^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: Add "native" as a valid option value for -march= on arm (PR driver/83193). 2018-02-22 12:16 ` Martin Liška @ 2018-03-08 9:02 ` Martin Liška 2018-03-09 15:43 ` Kyrill Tkachov 1 sibling, 0 replies; 21+ messages in thread From: Martin Liška @ 2018-03-08 9:02 UTC (permalink / raw) To: Kyrill Tkachov, gcc-patches Cc: Jakub Jelinek, Jan Hubicka, James Greenhalgh, Ramana Radhakrishnan PING^1 On 02/22/2018 01:15 PM, Martin LiÅ¡ka wrote: > On 02/22/2018 11:59 AM, Kyrill Tkachov wrote: >> Hi Martin, >> >> On 22/02/18 08:51, Martin LiÅ¡ka wrote: >>> On 02/21/2018 10:23 AM, Kyrill Tkachov wrote: >>>> On arm we also support "native" as a value for -mcpu and -mtune. These are both handled by >>>> the arm_print_hint_for_cpu_option function in the same file. Can you please add this snippet >>>> to them as well? >>> Hi. >>> >>> Can you please test for me attached follow up patch? >> >> I've bootstrapped and tested this patch on a native and a cross compiler and confirmed >> the help string for -mcpu behaves as intended. >> >> This patch is ok for trunk. >> Thanks, >> Kyrill > > Thanks for review. May I please ask you for taking look at: > 2b) point in: > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83193#c6 > > We should fix it (if possible) for GCC 8 as it's regression. > > Thanks, > Martin > >> >>> Thanks, >>> Martin >> > ^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: Add "native" as a valid option value for -march= on arm (PR driver/83193). 2018-02-22 12:16 ` Martin Liška 2018-03-08 9:02 ` Martin Liška @ 2018-03-09 15:43 ` Kyrill Tkachov 2018-03-09 15:46 ` Martin Liška 1 sibling, 1 reply; 21+ messages in thread From: Kyrill Tkachov @ 2018-03-09 15:43 UTC (permalink / raw) To: Martin Liška, gcc-patches Cc: Jakub Jelinek, Jan Hubicka, James Greenhalgh, Ramana Radhakrishnan On 22/02/18 12:15, Martin LiÅ¡ka wrote: > On 02/22/2018 11:59 AM, Kyrill Tkachov wrote: >> Hi Martin, >> >> On 22/02/18 08:51, Martin LiÅ¡ka wrote: >>> On 02/21/2018 10:23 AM, Kyrill Tkachov wrote: >>>> On arm we also support "native" as a value for -mcpu and -mtune. These are both handled by >>>> the arm_print_hint_for_cpu_option function in the same file. Can you please add this snippet >>>> to them as well? >>> Hi. >>> >>> Can you please test for me attached follow up patch? >> I've bootstrapped and tested this patch on a native and a cross compiler and confirmed >> the help string for -mcpu behaves as intended. >> >> This patch is ok for trunk. >> Thanks, >> Kyrill > Thanks for review. May I please ask you for taking look at: > 2b) point in: > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83193#c6 > > We should fix it (if possible) for GCC 8 as it's regression. Thanks for the reminders Martin. I've committed a patch to fix it with r258389. Kyrill > Thanks, > Martin > >>> Thanks, >>> Martin ^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: Add "native" as a valid option value for -march= on arm (PR driver/83193). 2018-03-09 15:43 ` Kyrill Tkachov @ 2018-03-09 15:46 ` Martin Liška 0 siblings, 0 replies; 21+ messages in thread From: Martin Liška @ 2018-03-09 15:46 UTC (permalink / raw) To: Kyrill Tkachov, gcc-patches Cc: Jakub Jelinek, Jan Hubicka, James Greenhalgh, Ramana Radhakrishnan On 03/09/2018 04:43 PM, Kyrill Tkachov wrote: > > On 22/02/18 12:15, Martin LiÅ¡ka wrote: >> On 02/22/2018 11:59 AM, Kyrill Tkachov wrote: >>> Hi Martin, >>> >>> On 22/02/18 08:51, Martin LiÅ¡ka wrote: >>>> On 02/21/2018 10:23 AM, Kyrill Tkachov wrote: >>>>> On arm we also support "native" as a value for -mcpu and -mtune. These are both handled by >>>>> the arm_print_hint_for_cpu_option function in the same file. Can you please add this snippet >>>>> to them as well? >>>> Hi. >>>> >>>> Can you please test for me attached follow up patch? >>> I've bootstrapped and tested this patch on a native and a cross compiler and confirmed >>> the help string for -mcpu behaves as intended. >>> >>> This patch is ok for trunk. >>> Thanks, >>> Kyrill >> Thanks for review. May I please ask you for taking look at: >> 2b) point in: >> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83193#c6 >> >> We should fix it (if possible) for GCC 8 as it's regression. > > Thanks for the reminders Martin. > I've committed a patch to fix it with r258389. I thank you for taking care of the ARM-specific issue. Let me adjust milestone of the PR to 9. The rest of issue seen can't be fixed now. Martin > > Kyrill > >> Thanks, >> Martin >> >>>> Thanks, >>>> Martin > ^ permalink raw reply [flat|nested] 21+ messages in thread
* [PATCH] Add "native" as a valid option value for -march= on aarch64 (PR driver/83193). 2018-02-20 13:25 [PATCH] Add "native" as a valid option value for -march= on i386 (PR driver/83193) Martin Liška 2018-02-20 15:58 ` Richard Sandiford 2018-02-21 7:34 ` Add "native" as a valid option value for -march= on arm " Martin Liška @ 2018-02-21 7:35 ` Martin Liška 2 siblings, 0 replies; 21+ messages in thread From: Martin Liška @ 2018-02-21 7:35 UTC (permalink / raw) To: gcc-patches Cc: Jakub Jelinek, Jan Hubicka, James Greenhalgh, Ramana Radhakrishnan, Kyrill Tkachov [-- Attachment #1: Type: text/plain, Size: 139 bytes --] Hi. This is equivalent patch for aarch64 target. I've tested that both as cross compiler and native one. Ready for trunk? Thanks, Martin [-- Attachment #2: 0002-Add-native-as-a-valid-option-value-for-march-on-aarc.patch --] [-- Type: text/x-patch, Size: 1850 bytes --] From 4fbe17099f8618ddd6a4de2d269ecb6f99625927 Mon Sep 17 00:00:00 2001 From: marxin <mliska@suse.cz> Date: Tue, 20 Feb 2018 14:14:25 +0100 Subject: [PATCH 2/3] Add "native" as a valid option value for -march= on aarch64 (PR driver/83193). gcc/ChangeLog: 2018-02-20 Martin Liska <mliska@suse.cz> PR driver/83193 * config/aarch64/aarch64.c (aarch64_print_hint_for_core_or_arch): Add "native" as a possible value. * config/aarch64/aarch64.h (HAVE_LOCAL_CPU_DETECT): Define the macro when native cpu detection is available. --- gcc/config/aarch64/aarch64.c | 7 +++++++ gcc/config/aarch64/aarch64.h | 1 + 2 files changed, 8 insertions(+) diff --git a/gcc/config/aarch64/aarch64.c b/gcc/config/aarch64/aarch64.c index e1fb87f047f..33c90ef02dc 100644 --- a/gcc/config/aarch64/aarch64.c +++ b/gcc/config/aarch64/aarch64.c @@ -10628,6 +10628,13 @@ aarch64_print_hint_for_core_or_arch (const char *str, bool arch) const struct processor *entry = arch ? all_architectures : all_cores; for (; entry->name != NULL; entry++) candidates.safe_push (entry->name); + +#ifdef HAVE_LOCAL_CPU_DETECT + /* Add also "native" as possible value. */ + if (arch) + candidates.safe_push ("native"); +#endif + char *s; const char *hint = candidates_list_and_hint (str, s, candidates); if (hint) diff --git a/gcc/config/aarch64/aarch64.h b/gcc/config/aarch64/aarch64.h index e3c52f63683..976f9afae54 100644 --- a/gcc/config/aarch64/aarch64.h +++ b/gcc/config/aarch64/aarch64.h @@ -1002,6 +1002,7 @@ extern const char *aarch64_rewrite_mcpu (int argc, const char **argv); #if defined(__aarch64__) extern const char *host_detect_local_cpu (int argc, const char **argv); +#define HAVE_LOCAL_CPU_DETECT # define EXTRA_SPEC_FUNCTIONS \ { "local_cpu_detect", host_detect_local_cpu }, \ MCPU_TO_MARCH_SPEC_FUNCTIONS -- 2.16.1 ^ permalink raw reply [flat|nested] 21+ messages in thread
end of thread, other threads:[~2018-03-09 15:46 UTC | newest] Thread overview: 21+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2018-02-20 13:25 [PATCH] Add "native" as a valid option value for -march= on i386 (PR driver/83193) Martin Liška 2018-02-20 15:58 ` Richard Sandiford 2018-02-20 17:49 ` Jakub Jelinek 2018-02-20 16:19 ` Richard Sandiford 2018-02-20 16:26 ` Martin Liška 2018-02-21 7:33 ` Martin Liška 2018-02-21 19:49 ` Jakub Jelinek 2018-02-21 14:19 ` Martin Liška 2018-02-21 14:23 ` Jakub Jelinek 2018-02-21 7:34 ` Add "native" as a valid option value for -march= on arm " Martin Liška 2018-02-21 9:24 ` Kyrill Tkachov 2018-02-21 9:53 ` Martin Liška 2018-02-21 12:26 ` Kyrill Tkachov 2018-02-21 14:03 ` Martin Liška 2018-02-22 8:52 ` Martin Liška 2018-02-22 10:59 ` Kyrill Tkachov 2018-02-22 12:16 ` Martin Liška 2018-03-08 9:02 ` Martin Liška 2018-03-09 15:43 ` Kyrill Tkachov 2018-03-09 15:46 ` Martin Liška 2018-02-21 7:35 ` [PATCH] Add "native" as a valid option value for -march= on aarch64 " Martin Liška
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).