<br><div class="gmail_extra"><br><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">
> This patch will emit a warning if the alignment is actually clamped.<br>
> It is hard to say whether we should emit a warning or throw a fatal error.<br>
<br>
</div></div>IMO, it should be an error; users will never see the message otherwise.<br>
<div class="im"><br>
> When creating stack object, we use Align<br>
> Align = std::max((unsigned)TLI.getDataLayout()->getPrefTypeAlignment(Ty),<br>
>                    AI->getAlignment());<br>
> The preferred alignment may be too big for the target, and it makes sense to give a warning instead of a fatal error.<br>
<br>
</div>If we can't realign the stack, we should print an error if<br>
AI->getAlignment() is too large and simply ignore the return of<br>
getPrefTypeAlignment if it's too large.</blockquote><div><br></div><div>+1000</div><div><br></div><div>There's no knowing what is depending upon the requested alignment and so this could silently break code.</div>
<div><br></div><div>-eric </div></div></div>