================
Comment at: cpp11-migrate/LoopConvert/VariableNaming.cpp:111
@@ +110,3 @@
+ while(DeclAncestor != NULL) {
+ if (const TemplateDecl *T = dyn_cast<TemplateDecl>(DeclAncestor)) {
+ // Check that the symbol is not a template parameter of the current
----------------
I think we could get away with just using DeclContexts directly and not making
a whole DeclAncestor map. The information being stored in that map is just a
replication of what's available by moving up the DeclContext chain.
================
Comment at: cpp11-migrate/LoopConvert/VariableNaming.cpp:89
@@ +88,3 @@
+ const Stmt *StmtAncestor = ReverseAST->lookup(StmtChild);
+ while (StmtAncestor != NULL) {
+ for (Stmt::const_child_iterator I = StmtAncestor->child_begin(),
----------------
It seems like there must be a better solution using DeclContexts to handle this
problem of CompoundStmts adding extra scopes.
http://llvm-reviews.chandlerc.com/D950
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
