[PATCH] D152688: [Aarch64] Add Cortex-A510 specific scheduling

Mon Jun 19 08:19:52 PDT 2023

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

OK, that sounds good. It would be good to have some tests for UNDEF nodes if we can, so we can add those separately. From what you have shown the performance has sounded pretty decent for SVE code, so scheduling of those UNDEF nodes should be quite important. Lets get this in and we can iterator from there.

LGTM, with a small Nit.

================
Comment at: llvm/lib/Target/AArch64/AArch64SchedA510.td:9
+//
+//
+//===----------------------------------------------------------------------===//
----------------
You can add "This file defines the machine model for the ARM Cortex-A510 processors."

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D152688