Merge pull request #29556 from rintaro/5.2-revert-sourcekit-rdar58550697

[5.2]Revert: [SourceKit] Disable module system headers validation

Author: rintaro

tools/SourceKit/lib/SwiftLang/SwiftASTManager.cpp

@@ -539,11 +539,6 @@ bool SwiftASTManager::initCompilerInvocation(
   // We don't care about LLVMArgs
   FrontendOpts.LLVMArgs.clear();
 
-  // This validation may call stat(2) many times. Disable it to prevent
-  // performance issues.
-  Invocation.getSearchPathOptions().DisableModulesValidateSystemDependencies =
-      true;
-
   // SwiftSourceInfo files provide source location information for decls coming
   // from loaded modules. For most IDE use cases it either has an undesirable
   // impact on performance with no benefit (code completion), results in stale

tools/swift-ide-test/swift-ide-test.cpp

@@ -3302,8 +3302,6 @@ int main(int argc, char *argv[]) {
     options::DebugForbidTypecheckPrefix;
   InitInvok.getTypeCheckerOptions().DebugConstraintSolver =
       options::DebugConstraintSolver;
-  InitInvok.getSearchPathOptions().DisableModulesValidateSystemDependencies =
-      true;
 
   for (auto ConfigName : options::BuildConfigs)
     InitInvok.getLangOptions().addCustomConditionalCompilationFlag(ConfigName);