[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
Thu Apr 29 16:18:18 PDT 2021
xbolva00 added a comment.
>> I've removed these warnings from the diagnostic groups -Wextra
But to remove all false positives, you need reports from users of -Wextra. Otherwise this would be dead code with no users.
I suggest to leave it in groups. Now you can ask folks to try this patch with kernel, Firefox, Chrome, XYZ and of course LLVM. Also you can try gcc’s testcases for this warning. If it is OK, I see no reasons to remove it from those groups.
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