[PATCH] D69017: Include sanitize blacklist and other extra deps as part of scan-deps output

2019-10-16 Thread Kousik Kumar via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG962ca076e51c: Include sanitize blacklist and other extra deps as part of scan-deps output (authored by kousikk). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D69017: Include sanitize blacklist and other extra deps as part of scan-deps output

2019-10-16 Thread Jan Korous via Phabricator via cfe-commits
jkorous accepted this revision. jkorous added a comment. Looks great! Thanks! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69017/new/ https://reviews.llvm.org/D69017 ___ cfe-commits mailing list

[PATCH] D69017: Include sanitize blacklist and other extra deps as part of scan-deps output

2019-10-16 Thread Michael Spencer via Phabricator via cfe-commits
Bigcheese accepted this revision. Bigcheese added a comment. This revision is now accepted and ready to land. lgtm. Jan may want to take a look as I believe he was looking at a related issue. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D69017: Include sanitize blacklist and other extra deps as part of scan-deps output

2019-10-15 Thread Kousik Kumar via Phabricator via cfe-commits
kousikk updated this revision to Diff 225153. kousikk added a comment. Lint fixes Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69017/new/ https://reviews.llvm.org/D69017 Files: clang/lib/Tooling/DependencyScanning/DependencyScanningWorker.cpp

[PATCH] D69017: Include sanitize blacklist and other extra deps as part of scan-deps output

2019-10-15 Thread Kousik Kumar via Phabricator via cfe-commits
kousikk created this revision. kousikk added reviewers: arphaman, dexonsmith, Bigcheese, jkorous. Herald added a project: clang. Herald added a subscriber: cfe-commits. kousikk updated this revision to Diff 225153. kousikk added a comment. Lint fixes Clang's -M mode includes these extra