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

2021-04-28 Thread Dmitry Babokin via Phabricator via cfe-commits
dbabokin added a comment. > Thanks for the report. It seems the thing to do is to modify the warning so > that variables marked extern are not candidates for this warning. I will make > a new commit with that modification unless there are other suggestions. I agree, this seems to be the right f

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

2021-04-27 Thread Dmitry Babokin via Phabricator via cfe-commits
dbabokin added a comment. One more false-positive: void foo() { extern int yydebug; yydebug = 1; } It was triggered in ISPC build. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D100581/new/ https://reviews.llvm.org/D100581 ___