[cfe-commits] r70497 - in /cfe/trunk: lib/AST/ASTContext.cpp lib/AST/ExprConstant.cpp test/Sema/attr-aligned.c

[Douglas Gregor]

On Apr 30, 2009, at 2:16 PM, Eli Friedman <eli.friedman at gmail.com>  
wrote:

> On Thu, Apr 30, 2009 at 10:32 AM, Douglas Gregor <dgregor at apple.com>  
> wrote:
>> +  // FIXME: Why do we ask for the preferred alignment?
>
> Because that's what __alignof is supposed to return?  Take a look at
> the output for __alignof(double) on x86-32 Linux, for example.

I didn't see that in the docs, but okay. I'll remove the FIXME.

> Generally, the change looks fine.

Thanks for reviewing.

   - Doug