[PATCH] D157514: [ARM] Set preferred function alignment
Dave Green via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Aug 16 07:47:16 PDT 2023
dmgreen accepted this revision.
dmgreen added a comment.
This revision is now accepted and ready to land.
I agree it makes sense to use the same alignments, especially for cortex-m cpus. Can you update the LoopAlignment. Maybe call it "CodeAlignment" now? I'm not sure that's better or not to change the name. The documentation can be changed to: "/// What alignment is preferred for loop bodies <and functions>, in log2(bytes)."
Otherwise LGTM. Thanks
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D157514/new/
https://reviews.llvm.org/D157514
More information about the llvm-commits
mailing list