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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits