[cfe-commits] [PATCH] ObjC: interface decls pointing to implementation decls

Fariborz Jahanian fjahanian at apple.com
Mon Jul 20 11:26:27 PDT 2009


On Jul 20, 2009, at 11:18 AM, Argyrios Kyrtzidis wrote:

>
> On Jul 20, 2009, at 8:28 AM, Fariborz Jahanian wrote:
>
>> I believe we have considered such approaches in the past and when  
>> it came to choices between
>> a little bit more cpu cycle and adding a new field to an AST node,  
>> we opted for little bit more of cpu cycles.
>
> This is actually less about making it easier on Sema, but more on  
> making this information available to other clients, preferably  
> without requiring them to duplicate stuff that Sema does.

My point was that you are adding a new data member to  
ObjCContainerDecl, among the most heavily used class in ObjC AST.
We have tried hard not to fatten such nodes in the past. But since  
this is more about a policy decision, I let Chris to
comment on this (BTW, I like you rational).

- Fariborz

>
>
> -Argiris
>




More information about the cfe-commits mailing list