[PATCH] D55665: Output to SARIF from scan-build

2018-12-13 Thread Aaron Ballman via Phabricator via cfe-commits
aaron.ballman closed this revision. aaron.ballman added a comment. Committed in r349082. I left the test file off; it looks like it might work, but given that scan-build puts its output into a temp directory, I wasn't able to see a good way to handle checking the output. Given the simplicity of

[PATCH] D55665: Output to SARIF from scan-build

2018-12-13 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. Thanks, looks perfectly sane to me! When i previously wanted to add tests for scan-build, it seemed fairly straightforward: diff --git a/test/Analysis/scan-build/lit.local.cfg b/test/Analysis/sc

[PATCH] D55665: Output to SARIF from scan-build

2018-12-13 Thread Aaron Ballman via Phabricator via cfe-commits
aaron.ballman created this revision. aaron.ballman added reviewers: NoQ, george.karpenkov. This updates the scan-build perl script to allow outputting to sarif in a more natural fashion by specifying `-sarif` as a command line argument, similar to how `-plist` is already supported. There appear