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

Aaron Ballman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Aug 20 12:36:42 PDT 2019


aaron.ballman added a comment.

In D65206#1606072 <https://reviews.llvm.org/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 post-commit. Continues to LGTM.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D65206/new/

https://reviews.llvm.org/D65206





More information about the cfe-commits mailing list