[PATCH] D69963: Handling null AssumptionCache in simplifyCFG

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Dec 7 08:47:31 PST 2019


fhahn added a comment.

Thanks, I'll land the patch



================
Comment at: llvm/lib/Transforms/Utils/SimplifyCFG.cpp:2277
       // Register the new instruction with the assumption cache if necessary.
       if (auto *II = dyn_cast_or_null<IntrinsicInst>(N))
+        if (AC && match(II, m_Intrinsic<Intrinsic::assume>()))
----------------
This is not required the using match as below. I'll update it before committing


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D69963





More information about the llvm-commits mailing list