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

Dávid Bolvanský via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Apr 28 13:31:13 PDT 2021


xbolva00 added a comment.

In D100581#2723794 <https://reviews.llvm.org/D100581#2723794>, @nickdesaulniers wrote:

> In D100581#2723789 <https://reviews.llvm.org/D100581#2723789>, @mbenfield wrote:
>
>> But gcc's behavior is to not trigger here so maybe following it is best.
>
> Perhaps file a bug against GCC to find whether that was intentional or a bug?

Yes, the best solution. Also consider to check gcc’s test-suite - if they have this type of testcase, then this is wanted behaviour.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D100581



More information about the cfe-commits mailing list