[clang][deps] Move enabling system header deps from `clang-scan-deps` to `DependencyS...
authorJan Svoboda <jan_svoboda@apple.com>
Mon, 14 Jun 2021 10:12:23 +0000 (12:12 +0200)
committerJan Svoboda <jan_svoboda@apple.com>
Mon, 14 Jun 2021 10:23:33 +0000 (12:23 +0200)
This patch moves enabling system header deps from `clang-scan-deps` into the `DependencyScanning` library. This will make it easier to preserve semantics of the original TU command-line for modular dependencies (see D104036).

Reviewed By: arphaman

Differential Revision: https://reviews.llvm.org/D104033

clang/lib/Tooling/DependencyScanning/DependencyScanningWorker.cpp
clang/tools/clang-scan-deps/ClangScanDeps.cpp

index 0011a56..a2f9b1c 100644 (file)
@@ -177,6 +177,7 @@ public:
     if (Opts->Targets.empty())
       Opts->Targets = {deduceDepTarget(Compiler.getFrontendOpts().OutputFile,
                                        Compiler.getFrontendOpts().Inputs)};
+    Opts->IncludeSystemHeaders = true;
 
     switch (Format) {
     case ScanningOutputFormat::Make:
index 8a15750..49c4757 100644 (file)
@@ -502,8 +502,6 @@ int main(int argc, const char **argv) {
           AdjustedArgs.push_back("/clang:-o");
           AdjustedArgs.push_back("/clang:" + LastO);
         }
-        AdjustedArgs.push_back("-Xclang");
-        AdjustedArgs.push_back("-sys-header-deps");
 
         if (!HasResourceDir) {
           StringRef ResourceDir =