[PATCH] D105695: [clang][tooling] Accept Clang invocations with multiple jobs

2021-07-27 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 rG11ee699b3c81: [clang][tooling] Accept Clang invocations with multiple jobs (authored by jansvoboda11). Changed prior to commit: https://reviews.ll

[PATCH] D105695: [clang][tooling] Accept Clang invocations with multiple jobs

2021-07-27 Thread Jan Svoboda via Phabricator via cfe-commits
jansvoboda11 added a comment. In D105695#2905028 , @dexonsmith wrote: > Seeing the `-fembed-bitcode` case made me think of `-save-temps`. I think > this will work since `-x cpp-output` should return false for `isSrcFile()`... > but probably worth addin

[PATCH] D105695: [clang][tooling] Accept Clang invocations with multiple jobs

2021-07-26 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. Seeing the `-fembed-bitcode` case made me think of `-save-temps`. I think this will work since `-x cpp-output` should return false for `isSrcFile()`... but probably worth adding a test

[PATCH] D105695: [clang][tooling] Accept Clang invocations with multiple jobs

2021-07-26 Thread Jan Svoboda via Phabricator via cfe-commits
jansvoboda11 updated this revision to Diff 361630. jansvoboda11 added a comment. Formatting Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D105695/new/ https://reviews.llvm.org/D105695 Files: clang/include/clang/Tooling/Tooling.h clang/lib/Tooli

[PATCH] D105695: [clang][tooling] Accept Clang invocations with multiple jobs

2021-07-26 Thread Jan Svoboda via Phabricator via cfe-commits
jansvoboda11 requested review of this revision. jansvoboda11 added a comment. Requesting re-review, since there are two changes: - `-cc1` commands that don't read a source file are ignored (e.g. jobs generated by `-fembed-bitcode`), - test now have `-target arm64-apple-macosx11.0.0` to ensure th

[PATCH] D105695: [clang][tooling] Accept Clang invocations with multiple jobs

2021-07-26 Thread Jan Svoboda via Phabricator via cfe-commits
jansvoboda11 updated this revision to Diff 361626. jansvoboda11 added a comment. Also ignore cc1 jobs that have inputs that are not source files. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D105695/new/ https://reviews.llvm.org/D105695 Files: c