[PATCH] D69781: [analyzer] Add test directory for scan-build

2020-02-03 Thread Artem Dergachev via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG5521236a1807: [analyzer] Re-land 0aba69e "Add test directory for scan-build." (authored by dergachev.a). Changed prior to commit: https://reviews.llvm.org/D69781?vs=227815&id=242164#toc Repository: r

[PATCH] D69781: [analyzer] Add test directory for scan-build

2020-02-03 Thread Artem Dergachev via Phabricator via cfe-commits
NoQ reopened this revision. NoQ added a comment. This revision is now accepted and ready to land. I'll try to re-land this. I heard there were a lot more issues than just honoring `CLANG_INSTALL_SCANBUILD=NO` (which i'll do my best to test before pushing), so i'll monitor the buildbots carefully

[PATCH] D69781: [analyzer] Add test directory for scan-build

2019-11-06 Thread Nico Weber via Phabricator via cfe-commits
thakis added a comment. Also, you probably need to make clang/test/CMakeLists.txt depend on the install target, to make sure scan-build is in bin/. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69781/new/ https://reviews.llvm.org/D69781 ___

[PATCH] D69781: [analyzer] Add test directory for scan-build

2019-11-05 Thread Nico Weber via Phabricator via cfe-commits
thakis added a comment. If CLANG_INSTALL_SCANBUILD=NO, then this tries to run scan-build from PATH, where it might not exist. Does this need some lit feature to check if scan-build is being copied to the bin dir by the build system? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACT

[PATCH] D69781: [analyzer] Add test directory for scan-build

2019-11-04 Thread Devin Coughlin via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG0aba69eb1a01: [analyzer] Add test directory for scan-build. (authored by dcoughlin). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69781/new/ https://review

[PATCH] D69781: [analyzer] Add test directory for scan-build

2019-11-03 Thread Artem Dergachev via Phabricator via cfe-commits
NoQ accepted this revision. NoQ added a comment. This revision is now accepted and ready to land. Woohoo, that's much more tests than i made in D55683 ! Let's hope it works on Windows :/ Comment at: clang/test/Analysis/scan-build/plist_html_ou

[PATCH] D69781: [analyzer] Add test directory for scan-build

2019-11-03 Thread Devin Coughlin via Phabricator via cfe-commits
dcoughlin created this revision. dcoughlin added a reviewer: NoQ. Herald added subscribers: llvm-commits, cfe-commits, Charusso, dkrupp, donat.nagy, Szelethus, mikhail.ramalho, a.sidorin, delcypher, szepet, baloghadamsoftware, xazax.hun. Herald added projects: clang, LLVM. The static analyzer's