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