[cfe-commits] r47191 - in /cfe/trunk: AST/Decl.cpp include/clang/AST/Attr.h include/clang/AST/Decl.h
Chris Lattner
clattner at apple.com
Fri Feb 15 15:56:20 PST 2008
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
More information about the cfe-commits
mailing list