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

2019-08-27 Thread Joe Ranieri via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG3385c5cc4dfd: Fix a SARIF exporter crash with macro expansions (authored by jranieri-grammatech). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D65209/new/

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

2019-08-20 Thread Aaron Ballman via Phabricator via cfe-commits
aaron.ballman added a comment. In 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

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

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

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

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.