[PATCH] D128758: [RISCV] Refine known bits for READ_VLENB

[Philip Reames via Phabricator via llvm-commits]

reames added inline comments.


================
Comment at: llvm/lib/Target/RISCV/RISCVISelLowering.cpp:9399
+    if (MaxVLenB == MinVLenB)
+      Known = KnownBits::makeConstant(APInt(BitWidth, MinVLenB));
     break;
----------------
craig.topper wrote:
> Just to make sure I understand, given the Known.Zero that was already done. This is equivalent to Known.One.setBit(Log2_32(MinVLenB))?
Yep, would you prefer I write it that way?  


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

https://reviews.llvm.org/D128758