On 28 January 2015 at 10:01, Thomas Preud'homme <thomas.preudho...@arm.com> wrote: >> From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches- >> ow...@gcc.gnu.org] On Behalf Of Thomas Preud'homme >> >> Hi Andrew, >> >> cortex-a57-fma-steering.c is really meant to be autosufficient with >> aarch64_register_fma_steering being the only interface which is why I >> thought it was not worth the trouble. Now I'm not saying I'm against it >> as it do make this interface more explicit. >> >> I'm currently about to take a plane but I'll change it as soon as I arrive >> at work. > > My apologies for the delay, got busy when I arrived. > > Please find below (and in attachment for real this time) the updated patch and > ChangeLog entries. > > *** gcc/ChangeLog *** > > 2015-01-26 Thomas Preud'homme thomas.preudho...@arm.com > > * config.gcc: Add cortex-a57-fma-steering.o to extra_objs for > aarch64-*-*. > * config/aarch64/t-aarch64: Add a rule for cortex-a57-fma-steering.o. > * config/aarch64/aarch64.h (AARCH64_FL_USE_FMA_STEERING_PASS): Define. > (AARCH64_TUNE_FMA_STEERING): Likewise. > * config/aarch64/aarch64-cores.def: Set > AARCH64_FL_USE_FMA_STEERING_PASS for cores with dynamic steering of > FMUL/FMADD instructions. > * config/aarch64/aarch64.c (aarch64_register_fma_steering): Declare. > (aarch64_override_options): Include cortex-a57-fma-steering.h. Call > aarch64_register_fma_steering () if AARCH64_TUNE_FMA_STEERING is true. > * config/aarch64/cortex-a57-fma-steering.h: New file. > * config/aarch64/cortex-a57-fma-steering.c: Likewise.
OK but wait for stage-1 to open for general development before you commit it please. /Marcus
