This is an archive of the discontinued LLVM Phabricator instance.

[analyzer] Fix text range end columns in SARIF to be exclusive
ClosedPublic

Authored by jranieri-grammatech on Jul 24 2019, 6:22 AM.

Diff Detail

Event Timeline

Herald added a project: Restricted Project. · View Herald TranscriptJul 24 2019, 6:22 AM
aaron.ballman accepted this revision.Jul 30 2019, 5:29 AM

This LGTM, but I'd appreciate a second reviewer chiming in only because Joe is a coworker.

This revision is now accepted and ready to land.Jul 30 2019, 5:29 AM

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.

This revision was automatically updated to reflect the committed changes.