<div class="gmail_quote">On 15 December 2011 19:07, Eli Friedman <span dir="ltr"><<a href="mailto:eli.friedman@gmail.com">eli.friedman@gmail.com</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">On Thu, Dec 15, 2011 at 6:49 PM, Nick Lewycky <<a href="mailto:nlewycky@google.com">nlewycky@google.com</a>> wrote:<br>
> The attached patch adds a method to Basic/TargetInfo to return the suitable<br>
> alignment, above which any object is considered over-aligned. This value is<br>
> also equal to the amount of alignment you're guaranteed to get out of malloc<br>
> and equal to alignof(std::max_align_t).<br>
<br>
</div></div>Looks like some irrelevant whitespace changes snuck in.</blockquote><div><br></div><div>I don't think so? The change to PS3PPU is not functional, but I had to audit it for this change anyways and I'm not going to notice and not fix the 80-column violation and the misaligned string.</div>

<div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I haven't<br>
looked carefully at the values you've chosen for various targets, but<br>
otherwise looks okay.  What are you planning on using this for?<br></blockquote><div><br></div><div>I have a diagnostic in the works that warns about using global operator new to construct an over-aligned object. This is PR9527 (except that I'm warning on the site of the "new Cls" expr, not upon completion of the class).</div>

<div><br></div><div>I'm mostly concerned about getting the right value for the various targets.</div><div><br></div><div>Nick</div><div><br></div></div>