llvmorg-github-actions[bot] wrote:

<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-clang

Author: Cyndy Ishida (cyndyishida)

<details>
<summary>Changes</summary>

This internally broke an incremental build. Disable while it gets investigated.

This is partial revert of cf8597bd3b87 
resolves: rdar://188026923

---
Full diff: https://github.com/llvm/llvm-project/pull/225563.diff


1 Files Affected:

- (modified) clang/lib/DependencyScanning/DependencyScanningWorker.cpp (+3) 


``````````diff
diff --git a/clang/lib/DependencyScanning/DependencyScanningWorker.cpp 
b/clang/lib/DependencyScanning/DependencyScanningWorker.cpp
index c681bb93b83146..c6855f605e1712 100644
--- a/clang/lib/DependencyScanning/DependencyScanningWorker.cpp
+++ b/clang/lib/DependencyScanning/DependencyScanningWorker.cpp
@@ -427,6 +427,9 @@ createScanCompilerInvocation(const CompilerInvocation 
&Invocation,
       true;
   ScanInvocation->getHeaderSearchOpts().ModulesForceValidateUserHeaders = 
false;
 
+  // Avoid some checks and module map parsing when loading PCM files.
+  ScanInvocation->getPreprocessorOpts().ModulesCheckRelocated = false;
+
   // FIXME: Do this even with PCHs by marking the option as something like
   // "preprocessor benign" in LangOptions.def so that it passes the
   // compatibility checks in ASTReader.

``````````

</details>


https://github.com/llvm/llvm-project/pull/225563
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to