[clang] [Clang][Diagnose] Minimal support on emit-include-location in sarif mode (PR #170415)
Aaron Ballman via cfe-commits
cfe-commits at lists.llvm.org
Fri Dec 5 05:47:25 PST 2025
https://github.com/AaronBallman commented:
Thank you for the fix! I think you should add a release note to `clang/docs/ReleaseNotes.rst` so users know about the fixed crash. I think it probably also makes sense to add a test case to `clang/unittests/Basic/SarifTest.cpp` showing the behavior of include locations and demonstrating the fix.
https://github.com/llvm/llvm-project/pull/170415
More information about the cfe-commits
mailing list