On Nov 17, 2010, at 7:54 PM, Argyrios Kyrtzidis wrote:

>>> 
>>> ASTContext::getTypeInfo will calculate a ASTRecordLayout if one is not 
>>> available.
>>> It's debatable whether we will avoid it but better be on the safe side ?
>> 
>> When does this warning trigger?  IMO, it would make sense to only do it on 
>> function definitions, not prototypes (because if you just have a proto, you 
>> probably don't have the option to change it, and that way you don't get the 
>> warning in every translation unit).
> 
> Yes, only on function definitions.
> 
>> 
>> If it just triggers on definitions, the layout will have to be computed at 
>> codegen time, so it shouldn't add any cost to -O0 compiles.
> 
> That's true but I was thinking about non-codegen situations like indexing ?

*shrug* I still think it's worth it to be simple.

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

Reply via email to