[PATCH] D60846: [ValueTracking] Improve isKnowNonZero for Ints

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 18 00:49:23 PDT 2019


nikic added inline comments.


================
Comment at: llvm/test/Transforms/InstCombine/cttz.ll:12
+; CHECK:       non_zero:
+; CHECK-NEXT:    [[CTZ:%.*]] = call i64 @llvm.cttz.i64(i64 [[X]], i1 false), !range !0
+; CHECK-NEXT:    [[CTZ32:%.*]] = trunc i64 [[CTZ]] to i32
----------------
Hm, it looks like the optimization we want doesn't actually happen here? The `i1 false` should have changed to an `i1 true`.


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

https://reviews.llvm.org/D60846





More information about the llvm-commits mailing list