<div class="gmail_quote">On Thu, Jun 7, 2012 at 3:08 PM, David Sehr <span dir="ltr"><<a href="mailto:sehr@google.com" target="_blank">sehr@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Please disregard my last comment.  Implementing that will be a bit<br>
challenging, as stackAlign is a back end value rather than a clang<br>
TargetInfo value. Are there targets where stackAlign < SuitableAlign?<br></blockquote><div><br></div><div>I suspect so, and that's worrisome...</div><div><br></div><div>I'm not intimately familiar with stack alignment and Joerg's concerns, but it seems like a *more* aligned alloca would be fine, but a less aligned alloca would be bad. Why is it a bad thing to increase the alignment to SuitableAlign?</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class="HOEnZb"><font color="#888888"><br>
David<br>
</font></span><div class="im HOEnZb"><br>
On Thu, Jun 7, 2012 at 2:57 PM, David Sehr <<a href="mailto:sehr@google.com">sehr@google.com</a>> wrote:<br>
</div><div class="HOEnZb"><div class="h5">> Joerg,<br>
><br>
> So the interesting case is where stackAlignment < SuitableAlignment.<br>
> Would it be reasonable with you to align to the minimum of those two?<br>
><br>
> David<br>
><br>
> On Thu, Jun 7, 2012 at 2:40 PM, Joerg Sonnenberger<br>
> <<a href="mailto:joerg@britannica.bec.de">joerg@britannica.bec.de</a>> wrote:<br>
>> On Thu, Jun 07, 2012 at 02:23:13PM -0700, David Sehr wrote:<br>
>>> I have recently been debugging a failure with one of the tests from<br>
>>> the gcc torture test suite and discovered that gcc and llvm seem to<br>
>>> differ on whether __builtin_alloca is aligned more than 0mod4 (gcc<br>
>>> thinks it's aligned by what clang stores inTargetInfo::SuitableAlign).<br>
>><br>
>> It should ensure that the stack alignment invariance from the ABI is<br>
>> honored. I don't think it should provide a stricter alignment.<br>
>> Please be very careful when changing this to not violate the above.<br>
>><br>
>> Joerg<br>
>> _______________________________________________<br>
>> cfe-commits mailing list<br>
>> <a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
>> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
</div></div></blockquote></div><br>