[PATCH] D69439: [APInt] Introduce APIntOps::GetPositionOfMostSignificantDifferentBit()

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 25 11:30:42 PDT 2019


lebedev.ri added inline comments.


================
Comment at: llvm/unittests/ADT/APIntTest.cpp:2774
+
+  for (unsigned BitWidth = 8; BitWidth <= 8; ++BitWidth) {
+    for (unsigned V0 = 0; V0 < (1u << BitWidth); ++V0) {
----------------
RKSimon wrote:
> Doesn't this only run once for BitWidth = 8 ?
Oops, forgot to undo that..


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D69439





More information about the llvm-commits mailing list