[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]

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
>