[PATCH] D58650: Add support for computing "zext of value" in KnownBits. NFCI

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 28 08:44:20 PST 2019


nikic added inline comments.


================
Comment at: llvm/trunk/include/llvm/Support/KnownBits.h:120
+  /// With ExtendedBitsAreKnownZero=false the extended bits are set to unknown.
+  KnownBits zext(unsigned BitWidth, bool ExtendedBitsAreKnownZero) {
+    unsigned OldBitWidth = getBitWidth();
----------------
Might be clearer to separate this into a `zext` and an `anyext` method?


Repository:
  rL LLVM

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

https://reviews.llvm.org/D58650





More information about the llvm-commits mailing list