[PATCH] D65211: [analyzer] Update the SARIF exporter to SARIF 2.1

2019-08-27 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL370068: Update the SARIF exporter to SARIF 2.1 (authored by jranieri, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed prior to commit: https://reviews.llvm

[PATCH] D65211: [analyzer] Update the SARIF exporter to SARIF 2.1

2019-08-20 Thread Aaron Ballman via Phabricator via cfe-commits
aaron.ballman added a comment. In D65211#1606089 , @aaron.ballman wrote: > This LGTM, but I'd appreciate a second reviewer chiming in only because Joe > is a coworker. Sufficient time has passed that any concerns other reviewers have can be addressed

[PATCH] D65211: [analyzer] Update the SARIF exporter to SARIF 2.1

2019-07-30 Thread Aaron Ballman via Phabricator via cfe-commits
aaron.ballman accepted this revision. aaron.ballman added a comment. This revision is now accepted and ready to land. This LGTM, but I'd appreciate a second reviewer chiming in only because Joe is a coworker. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llv

[PATCH] D65211: [analyzer] Update the SARIF exporter to SARIF 2.1

2019-07-24 Thread Joe Ranieri via Phabricator via cfe-commits
jranieri-grammatech created this revision. jranieri-grammatech added reviewers: NoQ, george.karpenkov, aaron.ballman. Herald added subscribers: cfe-commits, Charusso, dkrupp, donat.nagy, Szelethus, mikhail.ramalho, a.sidorin, szepet, baloghadamsoftware, xazax.hun. Herald added a project: clang. jr