[PATCH] D129203: [AArch64] Use Neoverse N2 sched model as default for:

[Dave Green via Phabricator via llvm-commits]

dmgreen accepted this revision.
dmgreen added a comment.
This revision is now accepted and ready to land.

Thanks for the patch. It may be possible to write a quick test that the scheduling info for one of the new instructions.

Otherwise LGTM.



================
Comment at: llvm/lib/Target/AArch64/AArch64.td:1157-1160
+def : ProcessorModel<"neoverse-512tvb", NeoverseN2Model,
                      ProcessorFeatures.Neoverse512TVB, [TuneNeoverse512TVB]>;
-def : ProcessorModel<"neoverse-v1", CortexA57Model,
+def : ProcessorModel<"neoverse-v1", NeoverseN2Model,
                      ProcessorFeatures.NeoverseV1, [TuneNeoverseV1]>;
----------------
c-rhodes wrote:
> @dmgreen Is it ok to use the N2 as the default model for these cores given they're not Armv9 and only support SVEv1?
Yeah - I think that's OK. The model will just have more coverage than the core. It should always be a better fit than the A57 model. The extra info in the schedules will just be unused.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D129203/new/

https://reviews.llvm.org/D129203