[PATCH] D149940: [ValueTracking][NFC] Split off computeKnownBitsFromCmp from computeKnownBitsFromAssume

Caroline Tice via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri May 5 13:14:53 PDT 2023


cmtice added a comment.

This change causes build failures for builds that eliminate asserts:

llvm-project/llvm/lib/Analysis/ValueTracking.cpp:895:12: error: unused variable 'BitWidth' [-Werror,-Wunused-variable]

  unsigned BitWidth = Known.getBitWidth();
           ^

1 error generated.

(After your commit, BitWidth is only used in assert statements).


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D149940/new/

https://reviews.llvm.org/D149940



More information about the llvm-commits mailing list