[llvm] [VPlan] Remove legacy costing inside VPBlendRecipe::computeCost (PR #171846)

Ramkumar Ramachandra via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 11 07:37:20 PST 2025


================
@@ -7081,6 +7081,11 @@ static bool planContainsAdditionalSimplifications(VPlan &Plan,
           return true;
       }
 
+      // The legacy cost model costs non-header phis with a scalar VF as a phi,
+      // but scalar unrolled VPlans will have VPBlendRecipes which emit selects.
+      if (VF.isScalar() && isa<VPBlendRecipe>(&R))
----------------
artagnon wrote:

Hm, should we be using the stronger vputils::onlyFirstLaneUsed instead of VF.isScalar? Not sure if the current patch will result in mismatches?

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


More information about the llvm-commits mailing list