[PATCH] D131165: [AArch64] Fix and add A64FX scheduling resource/latency info

Fri Aug 5 00:30:44 PDT 2022

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

I don't know the details of this core, I am trusting you on that, but from an LLVM perspective this looks good. The tests are nice to have too, thanks for adding them. I have a small suggestion on reducing some of the duplicates, but otherwise LGTM.

================
Comment at: llvm/test/tools/llvm-mca/AArch64/A64FX/A64FX-sve-instructions.s:632
+fmls	z0.s, z1.s, z7.s[3]
+fmov	z0.d, #-0.12500000
+fmov	z0.d, #-0.13281250
----------------
It might be worth removing some of these duplicate fmov's. I've done the same for the N2 schedule tests in rG408378a0b3b0

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D131165