[PATCH] D129886: [clang] Add -fdiagnostics-format=sarif for SARIF diagnostics

2022-07-15 Thread Vaibhav Yenamandra via Phabricator via cfe-commits
vaibhav.y added a comment. Might be worth hiding it from `--help`, despite the instability warning. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D129886/new/ https://reviews.llvm.org/D129886 ___ cfe-comm

[PATCH] D129886: [clang] Add -fdiagnostics-format=sarif for SARIF diagnostics

2022-07-15 Thread Christopher Di Bella via Phabricator via cfe-commits
cjdb added a comment. Thanks for getting this started! To get this ready for submission, would you be able to add a test please? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D129886/new/ https://reviews.llvm.org/D129886 __

[PATCH] D129886: [clang] Add -fdiagnostics-format=sarif for SARIF diagnostics

2022-07-15 Thread Abraham Corea Diaz via Phabricator via cfe-commits
abrahamcd updated this revision to Diff 445077. abrahamcd retitled this revision from "Adds `-fdiagnostics-format=sarif` flag option and warning." to "[clang] Add -fdiagnostics-format=sarif for SARIF diagnostics". abrahamcd edited the summary of this revision. abrahamcd added a comment. Commit m