[PATCH] D101480: Revert "[Clang] -Wunused-but-set-parameter and -Wunused-but-set-variable"

2021-04-28 Thread Michael Benfield via Phabricator via cfe-commits
mbenfield created this revision. mbenfield added reviewers: aeubanks, george.burgess.iv, lebedev.ri, Abpostelnicu. Herald added subscribers: frasercrmck, luismarques, apazos, sameer.abuasal, pengfei, s.egerton, Jim, jocewei, PkmX, the_o, brucehoult, MartinMosbeck, rogfer01, edward-jones, zzheng,

[PATCH] D101480: Revert "[Clang] -Wunused-but-set-parameter and -Wunused-but-set-variable"

2021-04-28 Thread Andi via Phabricator via cfe-commits
Abpostelnicu accepted this revision. Abpostelnicu added a comment. This revision is now accepted and ready to land. Thank you for this! Also I think it’s very well that you want to implement this warning in clang, with a little bit of more polish it will be up for merger! Repository: rG LLVM

[PATCH] D101480: Revert "[Clang] -Wunused-but-set-parameter and -Wunused-but-set-variable"

2021-04-28 Thread Arthur Eubanks via Phabricator via cfe-commits
aeubanks added a comment. https://reviews.llvm.org/rG6d8d1338629ceeaf6f56dc9eabc72e1a63f05169 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D101480/new/ https://reviews.llvm.org/D101480 ___ cfe-commits ma