Typos 'conpute', 'allignof', and 'comput' in the comment.<div><br></div><div>I'm not really a big fan of ASTContext knowing about the semantics of the GNU alignof extension like this, but the change looks correct. Changing the comment to talk about semantic alignment versus ABI alignment would help (but that doesn't cover the following-references / not-following-references half).<br>
<div><br></div><div><div class="gmail_quote">On Wed, Aug 7, 2013 at 8:19 PM, Rafael Espíndola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@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="im">>> To quote Clang, 'alignof' applied to an expression is a GNU extension.<br>
>> Thus we should presumably do what GCC does -- C++ doesn't have a notion of<br>
>> the alignment of a declaration as distinct from the alignment of its type.<br>
><br>
><br>
> OK, so that's not strictly-speaking true, but it doesn't have a notion of<br>
> the alignment being increased because all globals happen to be 16-byte<br>
> aligned on the target system.<br>
<br>
</div>The attached patch should fix it. Is it OK?<br>
<br>
Cheers,<br>
Rafael<br>
</blockquote></div><br></div></div>