[PATCH] D127229: [clang][deps] Set -disable-free for module compilations

2022-06-08 Thread Ben Langmuir via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG7a72dca74a27: [clang][deps] Set -disable-free for module compilations (authored by benlangmuir). Repository: rG LLVM Github Monorepo CHANGES SINC

[PATCH] D127229: [clang][deps] Set -disable-free for module compilations

2022-06-08 Thread Jan Svoboda via Phabricator via cfe-commits
jansvoboda11 accepted this revision. jansvoboda11 added a comment. This revision is now accepted and ready to land. LGTM! Comment at: clang/lib/Tooling/DependencyScanning/DependencyScanningWorker.cpp:338 +// always true for a driver invocation. +

[PATCH] D127229: [clang][deps] Set -disable-free for module compilations

2022-06-08 Thread Ben Langmuir via Phabricator via cfe-commits
benlangmuir added inline comments. Comment at: clang/lib/Tooling/DependencyScanning/DependencyScanningWorker.cpp:338 +// always true for a driver invocation. +bool DisableFree = true; DependencyScanningActi

[PATCH] D127229: [clang][deps] Set -disable-free for module compilations

2022-06-08 Thread Jan Svoboda via Phabricator via cfe-commits
jansvoboda11 added inline comments. Comment at: clang/lib/Tooling/DependencyScanning/DependencyScanningWorker.cpp:338 +// always true for a driver invocation. +bool DisableFree = true; DependencyScanningAct

[PATCH] D127229: [clang][deps] Set -disable-free for module compilations

2022-06-07 Thread Ben Langmuir via Phabricator via cfe-commits
benlangmuir updated this revision to Diff 434916. benlangmuir added a comment. Attempt to fix Windows path issue in test. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D127229/new/ https://reviews.llvm.org/D127229 Files: clang/lib/Tooling/Depende

[PATCH] D127229: [clang][deps] Set -disable-free for module compilations

2022-06-07 Thread Ben Langmuir via Phabricator via cfe-commits
benlangmuir created this revision. benlangmuir added reviewers: jansvoboda11, Bigcheese. Herald added a project: All. benlangmuir requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. The command-line arguments for module builds are cc1 commands,