pcc added a comment.
This warning seems to have a lot of false positives on things like reference
arguments that are used as output parameters. For example here is a small
sample of output from a stage2 build of part of LLVM:
In file included from ../llvm/lib/BinaryFormat/Minidump.cpp:9:
In file included from ../llvm/include/llvm/BinaryFormat/Minidump.h:21:
In file included from ../llvm/include/llvm/ADT/BitmaskEnum.h:16:
../llvm/include/llvm/Support/MathExtras.h:822:9: warning: variable
'Overflowed' set but not used [-Wunused-but-set-variable]
bool &Overflowed = ResultOverflowed ? *ResultOverflowed : Dummy;
^
../llvm/include/llvm/Support/MathExtras.h:936:72: warning: parameter 'Result'
set but not used [-Wunused-but-set-parameter]
std::enable_if_t<std::is_signed<T>::value, T> MulOverflow(T X, T Y, T
&Result) {
^
In file included from ../llvm/lib/BinaryFormat/Minidump.cpp:9:
In file included from ../llvm/include/llvm/BinaryFormat/Minidump.h:22:
In file included from ../llvm/include/llvm/ADT/DenseMapInfo.h:20:
../llvm/include/llvm/ADT/StringRef.h:511:37: warning: parameter 'Result' set
but not used [-Wunused-but-set-parameter]
getAsInteger(unsigned Radix, T &Result) const {
^
../llvm/include/llvm/ADT/StringRef.h:522:37: warning: parameter 'Result' set
but not used [-Wunused-but-set-parameter]
getAsInteger(unsigned Radix, T &Result) const {
^
../llvm/include/llvm/ADT/StringRef.h:545:39: warning: parameter 'Result' set
but not used [-Wunused-but-set-parameter]
consumeInteger(unsigned Radix, T &Result) {
^
../llvm/include/llvm/ADT/StringRef.h:556:39: warning: parameter 'Result' set
but not used [-Wunused-but-set-parameter]
consumeInteger(unsigned Radix, T &Result) {
^
6 warnings generated.
Could you please take a look?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D100581/new/
https://reviews.llvm.org/D100581
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
