aaron.ballman added a comment. In https://reviews.llvm.org/D28467#645348, @malcolm.parsons wrote:
> http://lab.llvm.org:8011/builders/sanitizer-ppc64be-linux/builds/1122/steps/bootstrap%20clang/logs/stdio > > > /home/buildbots/ppc64be-sanitizer/sanitizer-ppc64be/build/llvm/lib/Analysis/ValueTracking.cpp:1116:17: > error: lambda capture 'BitWidth' is not used > [-Werror,-Wunused-lambda-capture] > auto KOF = [BitWidth](const APInt &KnownOne, unsigned ShiftAmt) { > ^ > > /home/buildbots/ppc64be-sanitizer/sanitizer-ppc64be/build/llvm/lib/Analysis/ValueTracking.cpp:1127:17: > error: lambda capture 'BitWidth' is not used > [-Werror,-Wunused-lambda-capture] > auto KZF = [BitWidth](const APInt &KnownZero, unsigned ShiftAmt) { > ^ > > /home/buildbots/ppc64be-sanitizer/sanitizer-ppc64be/build/llvm/lib/Analysis/ValueTracking.cpp:1131:17: > error: lambda capture 'BitWidth' is not used > [-Werror,-Wunused-lambda-capture] > auto KOF = [BitWidth](const APInt &KnownOne, unsigned ShiftAmt) { > ^ > 3 errors generated. > > > The warnings are correct. > > Should I remove the unused lambda captures or revert this changeset? The warnings look correct to me; I think you should remove the unused lambda captures (no need for a test or review since changes are NFC). Repository: rL LLVM https://reviews.llvm.org/D28467 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits