Re: [PATCH][AArch64] Fix default CPU configurations

2014-03-12 Thread Marcus Shawcroft
On 11 March 2014 11:48, Kyrill Tkachov wrote: >> - if test x$target_cpu_cname = x >> + if test x"$target_cpu_cname" != x >> >> I think the addition of quoting here is orthogonal to the issue you >> are fixing. There are several other references to target_cpu_cname in >> config.gcc none of which a

Re: [PATCH][AArch64] Fix default CPU configurations

2014-03-11 Thread Kyrill Tkachov
Hi Marcus, On 11/03/14 11:25, Marcus Shawcroft wrote: On 25 February 2014 10:08, Kyrill Tkachov wrote: Hi all, The problem solved in this patch is that when gcc is configured with --with-arch=armv8-a gcc will go into aarch64-arches.def, pick the representative CPU (Cortex-A53 for ARMv8-A) an

Re: [PATCH][AArch64] Fix default CPU configurations

2014-03-11 Thread Marcus Shawcroft
On 25 February 2014 10:08, Kyrill Tkachov wrote: > Hi all, > > The problem solved in this patch is that when gcc is configured with > --with-arch=armv8-a gcc will go into aarch64-arches.def, pick the > representative CPU (Cortex-A53 for ARMv8-A) and use that CPUs ISA flags. Now > that we specifie

Re: [PATCH][AArch64] Fix default CPU configurations

2014-03-11 Thread Kyrill Tkachov
Ping^2 http://gcc.gnu.org/ml/gcc-patches/2014-02/msg01487.html Kyrill On 05/03/14 09:37, Kyrill Tkachov wrote: Ping? On 25/02/14 10:08, Kyrill Tkachov wrote: Hi all, The problem solved in this patch is that when gcc is configured with --with-arch=armv8-a gcc will go into aarch64-arches.def

Re: [PATCH][AArch64] Fix default CPU configurations

2014-03-05 Thread Kyrill Tkachov
Ping? On 25/02/14 10:08, Kyrill Tkachov wrote: Hi all, The problem solved in this patch is that when gcc is configured with --with-arch=armv8-a gcc will go into aarch64-arches.def, pick the representative CPU (Cortex-A53 for ARMv8-A) and use that CPUs ISA flags. Now that we specified that Cort

Re: [PATCH][AArch64] Fix default CPU configurations

2014-02-25 Thread Kyrill Tkachov
On 25/02/14 10:08, Kyrill Tkachov wrote: 2014-02-25 Kyrylo Tkachov * config.gcc (aarch64*-*-*): Use ISA flags from aarch64-arches.def. Do not define target_cpu_default2 to generic. * config/aarch64/aarch64.h (TARGET_CPU_DEFAULT): Use generic cpu. * config/aarch64/aarc

[PATCH][AArch64] Fix default CPU configurations

2014-02-25 Thread Kyrill Tkachov
Hi all, The problem solved in this patch is that when gcc is configured with --with-arch=armv8-a gcc will go into aarch64-arches.def, pick the representative CPU (Cortex-A53 for ARMv8-A) and use that CPUs ISA flags. Now that we specified that Cortex-A53 has CRC and crypto though, this means t