Re: [Patch AArch64 3/3] Fix up for pr70133

[Kyrill Tkachov <kyrylo.tkachov@foss.arm.com>]

Hi all,

On 06/04/16 11:10, James Greenhalgh wrote:
> Hi,
>
> Having updated the way we parse and output extension strings, now we just
> need to wire up the native detection to use these new features.
>
> In doing some cleanup and rename I ended up fixing 8-spaces to tabs in
> about half the file. I've done the rest while I'm here to save us from
> some a mixed-style file.
>
> Bootstrapped on aarch64-none-linux-gnu, then tested with defaults and
> an explicit -march=native passed (on a system detected as
> cortex-a57+crypto, and again on a system detected as
> cortex-a72.cortex-a53+crypto). I also set up a dummy /proc/cpuinfo and
> used that to manually check the input data in pr70133.
>
> OK?

This looks good to me (but I can't approve).

Thanks,
Kyrill

> Thanks,
> James
>
> ---
> 2016-04-06  James Greenhalgh  <james.greenhalgh@arm.com>
>
> 	PR target/70133
>
> 	* config/aarch64/driver-aarch64.c
> 	(aarch64_get_extension_string_for_isa_flags): New.
> 	(arch_extension): Rename to...
> 	(aarch64_arch_extension): ...This.
> 	(ext_to_feat_string): Rename to...
> 	(aarch64_extensions): ...This.
> 	(aarch64_core_data): Keep track of architecture extension flags.
> 	(cpu_data): Rename to...
> 	(aarch64_cpu_data): ...This.
> 	(aarch64_arch_driver_info): Keep track of architecture extension
> 	flags.
> 	(get_arch_name_from_id): Rename to...
> 	(get_arch_from_id): ...This, change return type.
> 	(host_detect_local_cpu): Update and reformat for renames, handle
> 	extensions through common infrastructure.
>