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 cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits