On 23 June 2015 at 09:49, James Greenhalgh <james.greenha...@arm.com> wrote: > > Hi, > > The FMA steering pass should be enabled through the tuning structures > rather than be an intrinsic property of the core. This patch moves > the control of the pass to the tuning structures - turning it off for > everything other than a Cortex-A57 system (i.e. -mcpu=cortex-a57 > or -mcpu=cortex-a57.cortex-a53). > > Some CPU's share the cortexa57 tuning structs, but do not use this > steering pass. For those I've taken a copy of the cortexa57 tuning > structures and called it cortexa72. > > Tested with a compiler build and all known values of -mcpu to make sure > the pass runs in the expected configurations. > > OK? > > Thanks, > James > > --- > 2015-06-23 James Greenhalgh <james.greenha...@arm.com> > > * config/aarch64/aarch64.h (AARCH64_FL_USE_FMA_STEERING_PASS): Delete. > (aarch64_tune_flags): Likewise. > (AARCH64_TUNE_FMA_STEERING): Likewise. > * config/aarch64/aarch64-cores.def (cortex-a57): Remove reference > to AARCH64_FL_USE_FMA_STEERING_PASS. > (cortex-a57.cortex-a53): Likewise. > (cortex-a72): Use cortexa72_tunings. > (cortex-a72.cortex-a53): Likewise. > (exynos-m1): Likewise. > * config/aarch64/aarch64-protos.h (tune_params): Add > a field: extra_tuning_flags. > * config/aarch64/aarch64-tuning-flags.def: New. > * config/aarch64/aarch64-protos.h (AARCH64_EXTRA_TUNING_OPTION): New. > (aarch64_extra_tuning_flags): Likewise. > (aarch64_tune_params): Declare here. > * config/aarch64/aarch64.c (generic_tunings): Set extra_tuning_flags. > (cortexa53_tunings): Likewise. > (cortexa57_tunings): Likewise. > (thunderx_tunings): Likewise. > (xgene1_tunings): Likewise. > (cortexa72_tunings): New. > * config/aarch64/cortex-a57-fma-steering.c: Include aarch64-protos.h. > (gate): Check against aarch64_tune_params. > * config/aarch64/t-aarch64 (cortex-a57-fma-steering.o): Depend on > aarch64-protos.h. >
OK /Marcus
