[PATCH] D65654: [AArch64] Set preferred function alignment to 16 bytes on Neoverse N1

Dave Green via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 2 07:43:01 PDT 2019


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

Looks very sensible to me! Thanks



================
Comment at: llvm/lib/Target/AArch64/AArch64Subtarget.cpp:82
     break;
   case CortexA65:
     break;
----------------
Whilst you are here can you set this to 3 too, like the A53.


================
Comment at: llvm/lib/Target/AArch64/AArch64Subtarget.cpp:127
     break;
   case NeoverseE1:
+    break;
----------------
This one can be 3 too I think. Like the A53.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D65654





More information about the llvm-commits mailing list