[llvm] [VPlan] Simplify VPBlendRecipes to select instructions (PR #133993)
Florian Hahn via llvm-commits
llvm-commits at lists.llvm.org
Thu Jul 10 04:19:35 PDT 2025
================
@@ -7047,6 +7047,13 @@ static bool planContainsAdditionalSimplifications(VPlan &Plan,
if (isa<VPPartialReductionRecipe>(&R))
return true;
+ // VPBlendRecipes are converted to selects and may have been simplified.
+ using namespace VPlanPatternMatch;
+ if (match(&R, m_VPInstruction<Instruction::Select>(
----------------
fhahn wrote:
Argh I see, so we create multiple selects with the same underlying instruction and this is causing the issue.
As lowering blends to selects becomes mandatory, it would probably be better to do it in convertToConcreteRecipes, at least to start with, removing the cost-model issue?
https://github.com/llvm/llvm-project/pull/133993
More information about the llvm-commits
mailing list