Reverted: Allowing SourceEntityWalker to skip visiting subscript end location

Swift SVN r31456

Author: Denis Vnukov

Diff for: include/swift/IDE/SourceEntityWalker.h

@@ -107,9 +107,6 @@ class SourceEntityWalker {
 
   virtual bool shouldWalkIntoFunctionGenericParams() { return true; }
 
-  /// Returns true if the walker should visit subscript end location
-  virtual bool shouldWalkIntoSubscriptEndLoc() { return true; }
-
 protected:
   SourceEntityWalker() = default;
   SourceEntityWalker(const SourceEntityWalker &) = default;

Diff for: lib/IDE/SourceEntityWalker.cpp

@@ -226,7 +226,7 @@ std::pair<bool, Expr *> SemaAnnotator::walkToExprPre(Expr *E) {
     if (!SE->getIndex()->walk(*this))
       return { false, nullptr };
 
-    if ((SubscrD != nullptr) && SEWalker.shouldWalkIntoSubscriptEndLoc()) {
+    if (SubscrD) {
       if (!passReference(SubscrD, E->getEndLoc()))
         return { false, nullptr };
     }