On 4 February 2015 at 10:35, Matthew Wahab <matthew.wa...@arm.com> wrote: > Hello, > > The Cortex-A72 is an ARMv8 core with the same architectural features as the > Cortex-A57. This patch adds support for the command line option > -mcpu=cortex-a72 with the same effect as the -mcpu=cortex-a57 option, only > the name being different. It also adds support for the > -mcpu=cortex-a72.cortex-a53 big-little variant. > > Tested with check-gcc on aarch64-none-linux-gnu with no new failures. Tested > the new cpu options from the command line. > > Matthew > > gcc/ > 2015-02-04 Matthew Wahab <matthew.wa...@arm.com> > > * config/aarch64/aarch64-cores.def: Add cortex-a72 and > cortex-a72.cortex-a53. > * config/aarch64/aarch64-tune.md: Regenerate. > * doc/invoke.texi (AArch64 Options/-mtune): Add "cortex-a72".
Thanks. Anyone using this new option, -mcpu=cortex-a72, will need to have a binutils build which includes your related binutils change (which I just OK'd) because the aarch64 backend in its current form will inject a .cpu directive into the output .S. This dependency is a tad unpleasant. James G has a patch in the works that will change gcc behaviour such that we pass through a .arch xxx+feature+feature directive into GAS which will remove this particular dependency but I think that change will need to wait for stage-1 now. This patch is low risk and I'd like the -mcpu=cortex-a72 recognition support in for gcc5. OK, commit it. /Marcus
