[PATCH] D65209: [analyzer] Fix a SARIF exporter crash with macro expansions

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


aaron.ballman added a comment.

In D65209#1606075 <https://reviews.llvm.org/D65209#1606075>, @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/D65209/new/

https://reviews.llvm.org/D65209





More information about the cfe-commits mailing list