[PATCH] D69643: [clang][ScanDeps] Fix issue with multiple commands with the same input.

2019-10-31 Thread Michael Spencer via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rGd816d9bdc585: [clang][ScanDeps] Fix issue with multiple commands with the same input. (authored by Bigcheese). Changed prior to commit: https://reviews.llvm.org/D69643?vs=227178&id=227347#toc Repositor

[PATCH] D69643: [clang][ScanDeps] Fix issue with multiple commands with the same input.

2019-10-30 Thread Alex Lorenz via Phabricator via cfe-commits
arphaman accepted this revision. arphaman added a comment. This revision is now accepted and ready to land. LGTM, Thanks Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69643/new/ https://reviews.llvm.org/D69643 ___

[PATCH] D69643: [clang][ScanDeps] Fix issue with multiple commands with the same input.

2019-10-30 Thread Michael Spencer via Phabricator via cfe-commits
Bigcheese created this revision. Bigcheese added reviewers: arphaman, klimek. Bigcheese added a project: clang. Herald added a subscriber: dexonsmith. Previously, given a CompilationDatabase with two commands for the same source file we would report that file twice with the union of the dependenci