[PATCH] D142234: [LVI] Handle Intrinsic::ctlz

Antonio Frighetto via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 31 23:55:26 PST 2023


antoniofrighetto marked an inline comment as done.
antoniofrighetto added inline comments.


================
Comment at: llvm/lib/IR/ConstantRange.cpp:1700
+  } else {
+    if (Lower.ugt(Upper))
+      std::swap(Lower, Upper);
----------------
nikic wrote:
> Can Lower be larger than Upper for a non-wrapped set?
It seems it is possible with negative numbers as well (`[0,-7)`), non-wrapped set.


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

https://reviews.llvm.org/D142234



More information about the llvm-commits mailing list