[PATCH] D55792: Allow direct navigation to static analysis checker documentation through SARIF exports

Thu Dec 20 08:35:09 PST 2018

aaron.ballman added a comment.

In D55792#1334886 <https://reviews.llvm.org/D55792#1334886>, @Szelethus wrote:

> This is awesome. Please wait for either @NoQ or @george.karpenkov to have the final say.
>
> Edit: Maybe couple days I guess. I've been digging through these files quite a lot in the last 2 months, and I don't see anything wrong with this patch.

@dkrupp, @NoQ, @george.karpenkov -- any comments? I'll probably commit early tomorrow if there isn't further feedback (and I'm happy to deal with anything post-commit if it arises).

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

https://reviews.llvm.org/D55792