From: Argyrios Kyrtzidis <akyrtzi@gmail.com>
Date: Sat, 18 Jul 2009 08:49:56 +0000 (+0000)
Subject: Remove StmtLocResolver::VisitObjCIvarRefExpr, it was only there to avoid returning... 
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=21b01c7e481b9c243f98155f8ab0cf4302d25f2a;p=clang

Remove StmtLocResolver::VisitObjCIvarRefExpr, it was only there to avoid returning an implicit 'self' instead of the ivar.
Since implicit 'self' no longer has a source location, it's not needed. (plus we also want to check for a 'self' that is visible in source code)

git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@76296 91177308-0d34-0410-b5e6-96231b3b80d8
---

diff --git a/lib/Index/ResolveLocation.cpp b/lib/Index/ResolveLocation.cpp
index 72781c324c..ba8134fb88 100644
--- a/lib/Index/ResolveLocation.cpp
+++ b/lib/Index/ResolveLocation.cpp
@@ -79,7 +79,6 @@ public:
     : LocResolverBase(ctx, loc), Parent(parent) {}
 
   ASTLocation VisitDeclStmt(DeclStmt *Node);
-  ASTLocation VisitObjCIvarRefExpr(ObjCIvarRefExpr *Node);
   ASTLocation VisitStmt(Stmt *Node);
 };
 
@@ -119,12 +118,6 @@ ASTLocation StmtLocResolver::VisitDeclStmt(DeclStmt *Node) {
   return ASTLocation(Parent, Node);
 }
 
-ASTLocation StmtLocResolver::VisitObjCIvarRefExpr(ObjCIvarRefExpr *Node) {
-  assert(ContainsLocation(Node) &&
-         "Should visit only after verifying that loc is in range");
-  return ASTLocation(Parent, Node);
-}
-
 ASTLocation StmtLocResolver::VisitStmt(Stmt *Node) {
   assert(ContainsLocation(Node) &&
          "Should visit only after verifying that loc is in range");