[cfe-commits] r47191 - in /cfe/trunk: AST/Decl.cpp include/clang/AST/Attr.h include/clang/AST/Decl.h

[Chris Lattner]

On Feb 15, 2008, at 3:41 PM, Ted Kremenek wrote:

> Hi Anders,
>
> I think such a map should go in ASTContext, rather than be a global
> variable.  It would be nice to invoke a parser and reclaim all
> associated state (including attributes) when we are done with a
> translation unit.  In general, we should avoid global variables of any
> kind.

In this case, the decls themselves own the attributes, just like they  
did before.  It's an implementation detail that the global exists, but  
the ownership is the same as before.

-Chris