[llvm] [VPlan] Strip bad assert in VPTransformState::get (NFC) (PR #166145)

Florian Hahn via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 4 05:24:47 PST 2025


================
@@ -308,11 +308,6 @@ Value *VPTransformState::get(const VPValue *Def, bool NeedsScalar) {
   VPLane LastLane(IsSingleScalar ? 0 : VF.getFixedValue() - 1);
   // Check if there is a scalar value for the selected lane.
   if (!hasScalarValue(Def, LastLane)) {
-    // At the moment, VPWidenIntOrFpInductionRecipes, VPScalarIVStepsRecipes and
-    // VPExpandSCEVRecipes can also be a single scalar.
-    assert((isa<VPWidenIntOrFpInductionRecipe, VPScalarIVStepsRecipe,
-                VPExpandSCEVRecipe>(Def->getDefiningRecipe())) &&
-           "unexpected recipe found to be invariant");
     IsSingleScalar = true;
     LastLane = 0;
   }
----------------
fhahn wrote:

Is the whole block here still needed?If those are not handled via isSingleScalar, we should be able to remove them. The point of the assert was to opt-out some recipes and manually handled them here.

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


More information about the llvm-commits mailing list