[llvm] [FuncSpec] Query SCCPSolver in more places (PR #114964)

Hari Limaye via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 5 04:25:37 PST 2024


================
@@ -347,10 +352,10 @@ Constant *InstCostVisitor::visitPHINode(PHINode &I) {
 
     // Disregard self-references and dead incoming values.
     if (auto *Inst = dyn_cast<Instruction>(V))
-      if (Inst == &I || DeadBlocks.contains(I.getIncomingBlock(Idx)))
----------------
hazzlim wrote:

I've left this as is, as per discussion above. I have added an assert just before inserting into DeadBlocks that the Solver thinks the block is executable.

https://github.com/llvm/llvm-project/pull/114964


More information about the llvm-commits mailing list