Merge pull request #76503 from DougGregor/unused-ifconfig-walk

Remove unused shouldWalkInactiveConfigRegion

Author: DougGregor

include/swift/IDE/SourceEntityWalker.h

@@ -196,9 +196,6 @@ class SourceEntityWalker {
   /// This method is called when a Module is referenced in source.
   virtual bool visitModuleReference(ModuleEntity Mod, CharSourceRange Range);
 
-  /// Whether walk into the inactive region in a #if config statement.
-  virtual bool shouldWalkInactiveConfigRegion() { return false; }
-
   virtual bool shouldWalkIntoGenericParams() { return true; }
 
   /// Only walk the arguments of a macro, to represent the source as written.

lib/IDE/SourceEntityWalker.cpp

@@ -209,15 +209,8 @@ ASTWalker::PreWalkAction SemaAnnotator::walkToDeclPreProper(Decl *D) {
     if (Loc.isValid())
       NameLen = PrecD->getName().getLength();
 
-  } else if (auto *ICD = dyn_cast<IfConfigDecl>(D)) {
-    if (SEWalker.shouldWalkInactiveConfigRegion()) {
-      for (auto Clause : ICD->getClauses()) {
-        for (auto Member : Clause.Elements) {
-          Member.walk(*this);
-        }
-      }
-      return Action::SkipNode();
-    }
+  } else if (isa<IfConfigDecl>(D)) {
+    // Nothing to do here.
   } else if (auto *MD = dyn_cast<MacroExpansionDecl>(D)) {
     if (auto *macro =
             dyn_cast_or_null<MacroDecl>(MD->getMacroRef().getDecl())) {