[PATCH] D65206: [analyzer] Fix text range end columns in SARIF to be exclusive

2019-08-27 Thread Joe Ranieri via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG68a6a28ef835: Fix text range end columns in SARIF to be exclusive (authored by jranieri-grammatech). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D65206/new/

[PATCH] D65206: [analyzer] Fix text range end columns in SARIF to be exclusive

2019-08-20 Thread Aaron Ballman via Phabricator via cfe-commits
aaron.ballman added a comment. In D65206#1606072 , @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] D65206: [analyzer] Fix text range end columns in SARIF to be exclusive

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] D65206: [analyzer] Fix text range end columns in SARIF to be exclusive

2019-07-24 Thread Joe Ranieri via Phabricator via cfe-commits
jranieri-grammatech created this revision. jranieri-grammatech added reviewers: george.karpenkov, NoQ, 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. A