[PATCH] D131632: [clang] Enable output of SARIF diagnostics

[Abraham Corea Diaz via Phabricator via cfe-commits]

abrahamcd added a comment.

In D131632#3733068 <https://reviews.llvm.org/D131632#3733068>, @aaron.ballman wrote:

> Precommit CI found a relevant failure.

I think this was just from the unfinished unit test. I can go ahead and remove `clang/unittests/Frontend/SARIFDiagnosticTest.cpp` entirely if we don't plan to use unit tests and the FileCheck test in `clang/test/Frontend/sarif-diagnostics.cpp` is enough.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D131632