[PATCH] D157514: [ARM] Set preferred function alignment

[Nicholas Guy via Phabricator via llvm-commits]

NickGuy updated this revision to Diff 548931.
NickGuy added a comment.

In D157514#4573318 <https://reviews.llvm.org/D157514#4573318>, @dmgreen wrote:

> Can you add a test? Thanks.

Done and precommitted.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D157514

Files:
  llvm/lib/Target/ARM/ARMISelLowering.cpp
  llvm/test/CodeGen/ARM/preferred-function-alignment.ll


Index: llvm/test/CodeGen/ARM/preferred-function-alignment.ll
===================================================================
--- llvm/test/CodeGen/ARM/preferred-function-alignment.ll
+++ llvm/test/CodeGen/ARM/preferred-function-alignment.ll
@@ -6,7 +6,7 @@
 
 ; CHECK-LABEL: test
 ; ALIGN-GENERIC: .p2align 2
-; ALIGN: .p2align 1
+; ALIGN: .p2align 2
 
 define void @test() {
   ret void
Index: llvm/lib/Target/ARM/ARMISelLowering.cpp
===================================================================
--- llvm/lib/Target/ARM/ARMISelLowering.cpp
+++ llvm/lib/Target/ARM/ARMISelLowering.cpp
@@ -1612,6 +1612,7 @@
   PredictableSelectIsExpensive = Subtarget->getSchedModel().isOutOfOrder();
 
   setPrefLoopAlignment(Align(1ULL << Subtarget->getPrefLoopLogAlignment()));
+  setPrefFunctionAlignment(Align(4));
 
   setMinFunctionAlignment(Subtarget->isThumb() ? Align(2) : Align(4));
 


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D157514.548931.patch
Type: text/x-patch
Size: 889 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20230810/f127122d/attachment.bin>