On 8 August 2013 01:36, Richard Smith <[email protected]> wrote:
> Typos 'conpute', 'allignof', and 'comput' in the comment.

Fixed.

> 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).

AST suffers from being the place we have for sharing code between Sema
and Codegen. As you point out, "semantic" would be a bit vague as what
we compute is specific to alignof.

Cheers,
Rafael

Attachment: t.patch
Description: Binary data

_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to