[PATCH] D89976: [ValueTracking] add range limits for ctpop

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 22 14:03:21 PDT 2020


nikic accepted this revision.
nikic added a comment.
This revision is now accepted and ready to land.

LGTM



================
Comment at: llvm/lib/Analysis/ValueTracking.cpp:6465
+    // Maximum of set bits is the bit width.
+    Upper = II.getType()->getScalarSizeInBits() + 1;
+    break;
----------------
Use `Width` instead of `II.getType()->getScalarSizeInBits()`?


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

https://reviews.llvm.org/D89976



More information about the llvm-commits mailing list