[PATCH] D11249: Fix PR#24142

hfinkel at anl.gov hfinkel at anl.gov
Sat Jul 25 18:36:51 PDT 2015


hfinkel added a subscriber: hfinkel.

================
Comment at: include/llvm/Target/TargetFrameLowering.h:79
@@ +78,3 @@
+      SPAdj = RoundUpToAlignment(SPAdj, StackAlignment);
+      SPAdj = -SPAdj;
+    } else {
----------------
Can't you just write this as:

   SPAdj = -RoundUpToAlignment(-SPAdj, StackAlignment);

================
Comment at: test/CodeGen/ARM/align-sp-adjustment.ll:3
@@ +2,3 @@
+
+; CHECK: [sp, #2120]
+
----------------
How stable is this number?



http://reviews.llvm.org/D11249







More information about the llvm-commits mailing list