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

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

Reply via email to