[cfe-commits] r108212 - /cfe/trunk/include/clang/AST/RecursiveASTVisitor.h

Craig Silverstein csilvers2000 at yahoo.com
Mon Jul 12 16:30:43 PDT 2010


Author: csilvers
Date: Mon Jul 12 18:30:43 2010
New Revision: 108212

URL: http://llvm.org/viewvc/llvm-project?rev=108212&view=rev
Log:
We should not be recursing over the shadow definitions in UsingDecl --
they're not something the user typed (at least, not here).

Modified:
    cfe/trunk/include/clang/AST/RecursiveASTVisitor.h

Modified: cfe/trunk/include/clang/AST/RecursiveASTVisitor.h
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/AST/RecursiveASTVisitor.h?rev=108212&r1=108211&r2=108212&view=diff
==============================================================================
--- cfe/trunk/include/clang/AST/RecursiveASTVisitor.h (original)
+++ cfe/trunk/include/clang/AST/RecursiveASTVisitor.h Mon Jul 12 18:30:43 2010
@@ -1062,11 +1062,6 @@
 
 DEF_TRAVERSE_DECL(UsingDecl, {
     TRY_TO(TraverseNestedNameSpecifier(D->getTargetNestedNameDecl()));
-    // FIXME: should we be iterating over the shadows?
-    for (UsingDecl::shadow_iterator I = D->shadow_begin(), E = D->shadow_end();
-         I != E; ++I) {
-      TRY_TO(TraverseDecl(*I));
-    }
   })
 
 DEF_TRAVERSE_DECL(UsingDirectiveDecl, {





More information about the cfe-commits mailing list