[PATCH] D126796: [clang][driver] adds `-print-diagnostics`

Thu Jun 2 08:31:11 PDT 2022

aaron.ballman added a comment.

In D126796#3551996 <https://reviews.llvm.org/D126796#3551996>, @cjdb wrote:

> In D126796#3551834 <https://reviews.llvm.org/D126796#3551834>, @tschuett wrote:
>
>> In D126796#3551781 <https://reviews.llvm.org/D126796#3551781>, @cjdb wrote:
>>
>>> In D126796#3551312 <https://reviews.llvm.org/D126796#3551312>, @tschuett wrote:
>>>
>>>> Would a JSON dump help you for your tools?
>>>
>>> Do you think a JSON dump would be more appropriate? I was just going for something not unlike `--help` here.
>>
>> I thought that you need this for your diagnostics campaign.
>
> Ah, gotcha. The idea for this feature came about while @aaron.ballman was reviewing a draft of that RFC, but it's orthogonal to that. I can't immediately see how users would benefit from this outputting JSON, but open to ideas.

+1 -- I think JSON output would be a better feature for `diagtool`. My goal with proposing the idea @cjdb has graciously worked on is to make warnings more discoverable for users in Clang itself, so I don't think JSON output helps there.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D126796