[PATCH] D76149: [AssumeBundles] Use assume bundles in isKnownNonZero

[Tyker via Phabricator via llvm-commits]

Tyker added inline comments.


================
Comment at: llvm/lib/Analysis/AssumeBundleQueries.cpp:149
+      IntrinsicInst *II = cast_or_null<IntrinsicInst>(Elem.Assume);
+      if (!II || Elem.Index == AssumptionCache::ExprResultIdx)
+        continue;
----------------
fhahn wrote:
> Can the cache contain elements that are not assumes? I think ideally it would only contain entries with valid assume calls.
the assumption cache cannot hold valid pointers to non-assume instruction.
but it can hold nullptr if an assume was in cache and gets deleted.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D76149