| Message ID | a04da397-59b1-4425-a07e-3a4298dd1e50@arm.com |
|---|---|
| State | New |
| Headers | show |
| Series | aarch64: Fix SCHEDULER_IDENT for Cortex-A510 | expand |
Hi Richard, I committed this combined patch (with Cortex-A520) for trunk https://gcc.gnu.org/git/?p=gcc.git;a=commit;h=cab53aae43cf94171b01320c08302e47a5daa391 Am I ok to commit just the Cortex-A510 half into gcc-12 and gcc-13. Thanks, Richard Ball
On 25/04/2024 15:59, Richard Ball wrote: > Hi Richard, > > I committed this combined patch (with Cortex-A520) for trunk https://gcc.gnu.org/git/?p=gcc.git;a=commit;h=cab53aae43cf94171b01320c08302e47a5daa391 <https://gcc.gnu.org/git/?p=gcc.git;a=commit;h=cab53aae43cf94171b01320c08302e47a5daa391> > > Am I ok to commit just the Cortex-A510 half into gcc-12 and gcc-13. Yes, if that's the correct thing to do there. R. > > Thanks, > Richard Ball > ------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------ > *From:* Richard Ball > *Sent:* 12 March 2024 14:08 > *To:* gcc-patches@gcc.gnu.org <gcc-patches@gcc.gnu.org>; Richard Earnshaw <Richard.Earnshaw@arm.com>; Richard Sandiford <Richard.Sandiford@arm.com>; Marcus Shawcroft <Marcus.Shawcroft@arm.com> > *Subject:* [PATCH][GCC] aarch64: Fix SCHEDULER_IDENT for Cortex-A510 > > The SCHEDULER_IDENT for this CPU was incorrectly > set to cortexa55, which is incorrect. This can cause > sub-optimal asm to be generated. > > Ok for trunk? > > Can I also backport this to gcc-12 and gcc-13? > > gcc/ChangeLog: > PR target/114272 > * config/aarch64/aarch64-cores.def (AARCH64_CORE): > Change SCHEDULER_IDENT from cortexa55 to cortexa53 > for Cortex-A510.
diff --git a/gcc/config/aarch64/aarch64-cores.def b/gcc/config/aarch64/aarch64-cores.def index 7ebefcf543b6f84b3df22ab836728111b56fa76f..ea84dcb11a2c11fb7d8ada3b66b3caaa39f08daf 100644 --- a/gcc/config/aarch64/aarch64-cores.def +++ b/gcc/config/aarch64/aarch64-cores.def @@ -169,7 +169,7 @@ AARCH64_CORE("cortex-r82", cortexr82, cortexa53, V8R, (), cortexa53, 0x41, 0xd15 /* Armv9.0-A Architecture Processors. */ /* Arm ('A') cores. */ -AARCH64_CORE("cortex-a510", cortexa510, cortexa55, V9A, (SVE2_BITPERM, MEMTAG, I8MM, BF16), cortexa53, 0x41, 0xd46, -1) +AARCH64_CORE("cortex-a510", cortexa510, cortexa53, V9A, (SVE2_BITPERM, MEMTAG, I8MM, BF16), cortexa53, 0x41, 0xd46, -1) AARCH64_CORE("cortex-a520", cortexa520, cortexa55, V9_2A, (SVE2_BITPERM, MEMTAG), cortexa53, 0x41, 0xd80, -1)