<div dir="ltr">On 2 August 2013 13:57, Bill Wendling <span dir="ltr"><<a href="mailto:isanbard@gmail.com" target="_blank">isanbard@gmail.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div class="HOEnZb"><div class="h5">On Aug 2, 2013, at 12:57 PM, Nick Lewycky <<a href="mailto:nlewycky@google.com">nlewycky@google.com</a>> wrote:<br>
<br>
> On 1 August 2013 14:41, Bill Wendling <<a href="mailto:isanbard@gmail.com">isanbard@gmail.com</a>> wrote:<br>
> Author: void<br>
> Date: Thu Aug  1 16:41:02 2013<br>
> New Revision: 187617<br>
><br>
> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=187617&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=187617&view=rev</a><br>
> Log:<br>
> Use function attributes to indicate if we don't want to realign the stack.<br>
><br>
> Modified:<br>
>     cfe/trunk/lib/CodeGen/BackendUtil.cpp<br>
>     cfe/trunk/lib/CodeGen/CGCall.cpp<br>
><br>
> Modified: cfe/trunk/lib/CodeGen/BackendUtil.cpp<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/CodeGen/BackendUtil.cpp?rev=187617&r1=187616&r2=187617&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/CodeGen/BackendUtil.cpp?rev=187617&r1=187616&r2=187617&view=diff</a><br>


> ==============================================================================<br>
> --- cfe/trunk/lib/CodeGen/BackendUtil.cpp (original)<br>
> +++ cfe/trunk/lib/CodeGen/BackendUtil.cpp Thu Aug  1 16:41:02 2013<br>
> @@ -451,7 +451,6 @@ TargetMachine *EmitAssemblyHelper::Creat<br>
>    Options.UnsafeFPMath = CodeGenOpts.UnsafeFPMath;<br>
>    Options.UseSoftFloat = CodeGenOpts.SoftFloat;<br>
>    Options.StackAlignmentOverride = CodeGenOpts.StackAlignment;<br>
> -  Options.RealignStack = CodeGenOpts.StackRealignment;<br>
>    Options.DisableTailCalls = CodeGenOpts.DisableTailCalls;<br>
>    Options.TrapFuncName = CodeGenOpts.TrapFuncName;<br>
>    Options.PositionIndependentExecutable = LangOpts.PIELevel != 0;<br>
><br>
> Modified: cfe/trunk/lib/CodeGen/CGCall.cpp<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/CodeGen/CGCall.cpp?rev=187617&r1=187616&r2=187617&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/CodeGen/CGCall.cpp?rev=187617&r1=187616&r2=187617&view=diff</a><br>


> ==============================================================================<br>
> --- cfe/trunk/lib/CodeGen/CGCall.cpp (original)<br>
> +++ cfe/trunk/lib/CodeGen/CGCall.cpp Thu Aug  1 16:41:02 2013<br>
> @@ -1084,6 +1084,9 @@ void CodeGenModule::ConstructAttributeLi<br>
><br>
>      FuncAttrs.addAttribute("no-frame-pointer-elim-non-leaf",<br>
>                             llvm::toStringRef(NoFramePointerElimNonLeaf));<br>
> +<br>
> +    if (!CodeGenOpts.StackRealignment)<br>
> +      FuncAttrs.addAttribute("no-realign-stack");<br>
>    }<br>
><br>
> This is not what mstackrealign does. There are three stack realignment modes, on off and forced. -mstackrealign forces a stack alignment to occur on entry to every function. no-realign-stack (off) forbids realigning stack even if you put a stack object on the stack which requires a higher alignment than the stack provides.<br>


><br>
</div></div>The flag that I replaced was a boolean flag (see r187618). It was turned on by default and only turned off when the CodeGenOpts.StackRealignment value was turned `false.' From what I can tell, it didn't have a third state. However, I may have uncovered a bug of some sort. Send me a test case and I can adjust this for you. :)<br>

</blockquote><div><br></div><div>Understood. At the driver level there's -mstackrealign/[nothing]/-mno-stackrealign which maps to the cc1 flags "-mstackrealign -force-align-stack"/"-mstackrealign"/[nothing] respectively. Each of those cc1 flags is indeed a boolean.</div>

<div><br></div><div>After much analysis, we've determined that our stack misalignment bug is actually in libstdc++ and not triggered by your change at all, despite showing up at the exactly the same time. Sorry Bill! I owe you a beer!</div>

<div><br></div><div>Nick</div><div><br></div></div></div></div>