Thanks, r162297.<div><br></div><div>Is there a reason why llvm::RoundUpToAlignment uses a divide and a multiply, rather than an and? (Is it trying to handle non-power-of-two alignments?)<br><br><div class="gmail_quote">On Tue, Aug 21, 2012 at 9:52 AM, Jakob Stoklund Olesen <span dir="ltr"><<a href="mailto:stoklund@2pi.dk" target="_blank">stoklund@2pi.dk</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5"><br>
On Aug 21, 2012, at 9:36 AM, Jordan Rose <<a href="mailto:jordan_rose@apple.com">jordan_rose@apple.com</a>> wrote:<br>
<br>
><br>
> On Aug 20, 2012, at 22:42 , Richard Smith <<a href="mailto:richard-llvm@metafoo.co.uk">richard-llvm@metafoo.co.uk</a>> wrote:<br>
><br>
>> +    ArrayIndexSize = (ArrayIndexSize + Align - 1) & ~(Align - 1);<br>
><br>
> Yuck. Can we put this (and the other time in appears, during allocation) in a helper function in AlignOf.h?<br>
<br>
</div></div>MathExtras.h already has RoundUpToAlignment().<br>
<span class="HOEnZb"><font color="#888888"><br>
/jakob<br>
<br>
</font></span></blockquote></div><br></div>