[clang] [analyzer] Emit IssueHash in SARIF (PR #159445)

Ziqing Luo via cfe-commits cfe-commits at lists.llvm.org
Wed Sep 24 15:56:20 PDT 2025


https://github.com/ziqingluo-90 approved this pull request.

Looks good to me.  Moving `getIssueHash` out of `HTMLDiagnostics`  is good since it is not something HTML-specific.

If I have to nitpick, I would suggest to keep the directories that are relative to the llvm repo in `hostedViewerUri`. (I guess this might not be an trivial task with `sed` though?) 

https://github.com/llvm/llvm-project/pull/159445


More information about the cfe-commits mailing list