cjdb added a comment.
In D138939#3965985 <https://reviews.llvm.org/D138939#3965985>, @tschuett wrote:
> Then Sarif was a distraction. Still to reduce boilerplate and for A/B testing:
>
> enum class DiagnosticMode {
> Legacy,
> UserOriented,
> Default = Legacy
> }
It took a fair bit of squinting and coffee, but I think I get it now. Having
SARIF will be good for option 2: I hadn't realised this was at the Clang API
level and not an enum for users to toggle on the command line! Thanks for your
patience, I'll implement this now.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D138939/new/
https://reviews.llvm.org/D138939
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
