dbartol wrote:

> 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?)

It would be tricky, and I wasn't sure if the all of the file URLs would always 
point into the LLVM repo anyway. I could probably leave at least the last 
directory component, though. If it's OK, I'll attempt that in a future PR. I've 
got a bunch more SARIF-related PRs to open in the next few weeks.


https://github.com/llvm/llvm-project/pull/159445
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to