[PATCH] D100581: [Clang] -Wunused-but-set-parameter and -Wunused-but-set-variable

Wed Apr 28 12:28:48 PDT 2021

lebedev.ri added a comment.

In D100581#2723621 <https://reviews.llvm.org/D100581#2723621>, @Abpostelnicu wrote:

> In D100581#2723611 <https://reviews.llvm.org/D100581#2723611>, @lebedev.ri wrote:
>
>> In D100581#2723505 <https://reviews.llvm.org/D100581#2723505>, @Abpostelnicu wrote:
>>
>>> Also I don’t remember seeing this proposed on cfe dev mailing list.
>>
>> There is no such requirement. I don't recall that happening basically ever, actually.
>
> Of course, but I would have wanted to see this discussed on the mailing list since the pool of the affected actors by this is very big.

If we subtract the false-positives, i'm not really sure what should be discussed.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D100581