[cfe-dev] [PATCH] : Proper name lookup for namespaces

[Chris Lattner]

On May 6, 2008, at 7:06 PM, Argiris Kirtzidis wrote:

> Hi,
>
> The attached patch contains changes to support proper name lookup  
> for namespaces.
> It is mostly an overhaul of the IdentifierResolver:
>
> -It exposes an iterator interface to get all decls through the scope  
> chain:
> for (IdentifierResolver::iterator
>      I = IdResolver.begin(II, CurContext), E = IdResolver.end(II);  
> I != E; ++I)
>   if ((*I)->getIdentifierNamespace() & NS)
>     return *I;
>
> -The semantic staff (checking IdentifierNamespace and Doug's  
> checking for shadowed tags were moved out of IdentifierResolver and  
> back into Sema. IdentifierResolver just gives an iterator for  
> iterating over all reachable decls of an identifier.
>
> -Fixes bug: http://llvm.org/bugs/show_bug.cgi?id=2275

Looks great to me, thanks Argiris!

-Chris