cjdb added a subscriber: abrahamcd.
cjdb added a comment.

I think this CL (which is quite large) might be worth just getting the 
functionality in, and deferring the in-tree usage to a second CL. That second 
CL probably should just lift the existing diagnostics so that they're in the 
message component if the SARIF flag is enabled: nothing fancy. I also expect 
this to be fairly non-intrusive with respect to the compiler, but I expect that 
the diagnostic engine will need some teaching here.

We're intending @abrahamcd to pilot proper true integration, by getting 
overload resolution failure diagnostics to be SARIF-operational.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D109701/new/

https://reviews.llvm.org/D109701

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to