[PATCH] D109701: [clang] Emit SARIF Diagnostics: Create `clang::SarifDocumentWriter` interface

[Abraham Corea Diaz via Phabricator via cfe-commits]

abrahamcd added a comment.

In D109701#3648168 <https://reviews.llvm.org/D109701#3648168>, @vaibhav.y wrote:

> In D109701#3646856 <https://reviews.llvm.org/D109701#3646856>, @abrahamcd wrote:
>
>> Hi! I'm interning with @cjdb and @denik this summer and I was working on adding a `-fdiagnostics-format=sarif` option to start off my project, but I just found that a previous abandoned version of this change (D109697 <https://reviews.llvm.org/D109697>) was intending to add it. Seeing as the flag is no longer included in this version of the change, is it okay for me to go on and add it myself, or are you still planning on adding it here? Thanks!
>
> Sure, feel free to use D109697 <https://reviews.llvm.org/D109697> as you see fit!

Great, thank you!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D109701