[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