[PATCH] D11160: Targets: commonize some stack realignment code

JF Bastien jfb at chromium.org
Mon Jul 20 13:29:00 PDT 2015


jfb added inline comments.

================
Comment at: lib/CodeGen/TargetRegisterInfo.cpp:323
@@ +322,3 @@
+                              F->hasFnAttribute(Attribute::StackAlignment));
+  if (ForceStackAlign || requiresRealignment) {
+    if (canRealignStack(MF))
----------------
I discussed this offline with @rnk, and I'll submit a follow-up to make it a hard-failure (not just debug print). I'm making it a follow-up so it's less likely to get reverted.


http://reviews.llvm.org/D11160







More information about the llvm-commits mailing list