[PATCH] D145178: [clang][NFC] reformats the SARIF diagnostic test so it's human readable

Christopher Di Bella via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Mar 3 10:32:11 PST 2023


cjdb added a comment.

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

> LGTM, this is incremental progress. Hopefully we won't be adding too many more RUN lines to this file though (sticking too many tests into one file is also tech debt).

Thanks! 100% of the RUN lines I intend to add are bug regressions, but there will be more //RUN// lines in the future.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D145178



More information about the cfe-commits mailing list