[cfe-commits] r60993 - in /cfe/trunk: include/clang/AST/DeclObjC.h lib/AST/DeclObjC.cpp lib/Sema/SemaDecl.cpp lib/Sema/SemaDeclObjC.cpp

Fariborz Jahanian fjahanian at apple.com
Wed Dec 17 09:08:29 PST 2008

On Dec 17, 2008, at 6:34 AM, steve naroff wrote:

> On Dec 15, 2008, at 1:02 PM, Douglas Gregor wrote:
>>>> Will we need to have the RecordDecl layout for most or all
>>>> Objective-C interfaces? If most programs only need the layout for a
>>>> small number of interfaces, it might be beneficial to compute the
>>>> layouts lazily (as we do with the layout of RecordDecls).
>>> Lay out is needed when a field is referenced. I will look at this
>>> later when I am done with code gen.
>> Okay!
> Just to build on this issue...why does ObjCInterfaceDecl need a  
> 'RecordForDecl' slot at all?
> From my perspective, this should be computed lazily (as you suggest)  
> and stored separately (using a map that is populated during code  
> generation).

Storage lay out is done lazily. Build of the record decl can also be  
done lazily. Saving a slot ONTOH, may not be worth the extra  
complexity, but certainly
can be done as well.

- Fariborz

> snaroff
>> 	- Doug
>> _______________________________________________
>> cfe-commits mailing list
>> cfe-commits at cs.uiuc.edu
>> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

More information about the cfe-commits mailing list