[PATCH] D18471: [CodeGen] Don't assume that fixed stack objects are aligned in a stack-realigned function.

[Charles Davis via llvm-commits]

cdavis5x added inline comments.

================
Comment at: lib/CodeGen/MachineFunction.cpp:87-88
@@ -80,2 +86,4 @@
+      MachineFrameInfo(getFnStackAlignment(STI, Fn),
                        STI->getFrameLowering()->isStackRealignable(),
-                       !F->hasFnAttribute("no-realign-stack"));
+                       !F->hasFnAttribute("no-realign-stack"),
+                       !F->hasFnAttribute("no-realign-stack") &&
----------------
rnk wrote:
> As a separate issue, I noticed that StackRealignable and RealignOpt are redundant. We can get by with just StackRealignable, and making that be `STI->getFrameLowering()->isStackRealignable() && F->hasFnAttribute("no-realign-stack")`.
Yeah, but I think I'll save that for another patch.


Repository:
  rL LLVM

http://reviews.llvm.org/D18471