[PATCH] D76149: [AssumeBundles] Use assume bundles in isKnownNonZero
Johannes Doerfert via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Apr 3 14:05:55 PDT 2020
jdoerfert added a comment.
I'm fine with this, one test needs an update though.
================
Comment at: llvm/test/Transforms/Attributor/nonnull.ll:35
+ ret i8* %ret
+}
+
----------------
I don't think these test what they should (the change in the Attributor). Since the Attributor knows about assumes (D74888) it can do this deduction already: https://godbolt.org/z/S-PNc-
I think this should work as test though:
https://godbolt.org/z/_6mVi3
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D76149/new/
https://reviews.llvm.org/D76149
More information about the llvm-commits
mailing list