[PATCH] D104536: [clang][deps] Avoid minimizing PCH input files

2021-07-20 Thread Jan Svoboda via Phabricator via cfe-commits
jansvoboda11 added a comment. In D104536#288 , @thakis wrote: > Looks like this breaks tests on Windows: > http://45.33.8.238/win/42264/step_7.txt > > Please take a look, and revert for now if it takes a while to fix. Thanks, that was a bad rebase.

[PATCH] D104536: [clang][deps] Avoid minimizing PCH input files

2021-07-20 Thread Nico Weber via Phabricator via cfe-commits
thakis added a comment. Looks like this breaks tests on Windows: http://45.33.8.238/win/42264/step_7.txt Please take a look, and revert for now if it takes a while to fix. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D104536/new/ https://reviews.l

[PATCH] D104536: [clang][deps] Avoid minimizing PCH input files

2021-07-20 Thread Jan Svoboda 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 rGe564fd93ab85: [clang][deps] Avoid minimizing PCH input files (authored by jansvoboda11). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST A

[PATCH] D104536: [clang][deps] Avoid minimizing PCH input files

2021-07-16 Thread Duncan P. N. Exon Smith via Phabricator via cfe-commits
dexonsmith accepted this revision. dexonsmith added a comment. This revision is now accepted and ready to land. LGTM. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D104536/new/ https://reviews.llvm.org/D104536 ___ cfe-commits mailing list cfe-

[PATCH] D104536: [clang][deps] Avoid minimizing PCH input files

2021-07-16 Thread Jan Svoboda via Phabricator via cfe-commits
jansvoboda11 updated this revision to Diff 359299. jansvoboda11 added a comment. Rebase on top of D106146 . CHANGES SINCE LAST ACTION https://reviews.llvm.org/D104536/new/ https://reviews.llvm.org/D104536 Files: clang/lib/Tooling/DependencyScanning/Depende