[llvm] [VPlan] Add specialized VPValue subclasses for different types (NFC) (PR #172758)
via llvm-commits
llvm-commits at lists.llvm.org
Sun Dec 28 08:09:23 PST 2025
github-actions[bot] wrote:
<!--LLVM CODE FORMAT COMMENT: {clang-format}-->
:warning: C/C++ code formatter, clang-format found issues in your code. :warning:
<details>
<summary>
You can test this locally with the following command:
</summary>
``````````bash
git-clang-format --diff origin/main HEAD --extensions cpp,h -- llvm/lib/Transforms/Vectorize/LoopVectorizationPlanner.h llvm/lib/Transforms/Vectorize/LoopVectorize.cpp llvm/lib/Transforms/Vectorize/VPlan.cpp llvm/lib/Transforms/Vectorize/VPlan.h llvm/lib/Transforms/Vectorize/VPlanAnalysis.cpp llvm/lib/Transforms/Vectorize/VPlanConstruction.cpp llvm/lib/Transforms/Vectorize/VPlanPatternMatch.h llvm/lib/Transforms/Vectorize/VPlanRecipes.cpp llvm/lib/Transforms/Vectorize/VPlanTransforms.cpp llvm/lib/Transforms/Vectorize/VPlanUnroll.cpp llvm/lib/Transforms/Vectorize/VPlanUtils.cpp llvm/lib/Transforms/Vectorize/VPlanValue.h llvm/unittests/Transforms/Vectorize/VPlanTest.cpp --diff_from_common_commit
``````````
:warning:
The reproduction instructions above might return results for more than one PR
in a stack if you are using a stacked PR workflow. You can limit the results by
changing `origin/main` to the base branch/commit you want to compare against.
:warning:
</details>
<details>
<summary>
View the diff from clang-format here.
</summary>
``````````diff
diff --git a/llvm/lib/Transforms/Vectorize/VPlan.h b/llvm/lib/Transforms/Vectorize/VPlan.h
index 035c651cd..a999d9967 100644
--- a/llvm/lib/Transforms/Vectorize/VPlan.h
+++ b/llvm/lib/Transforms/Vectorize/VPlan.h
@@ -1717,9 +1717,9 @@ public:
: VPRecipeWithIRFlags(VPDef::VPWidenCallSC, CallArguments, Flags, DL),
VPIRMetadata(Metadata), Variant(Variant) {
setUnderlyingValue(UV);
- assert(isa<Function>(
- getOperand(getNumOperands() - 1)->getLiveInIRValue()) &&
- "last operand must be the called function");
+ assert(
+ isa<Function>(getOperand(getNumOperands() - 1)->getLiveInIRValue()) &&
+ "last operand must be the called function");
}
~VPWidenCallRecipe() override = default;
@@ -1739,8 +1739,7 @@ public:
VPCostContext &Ctx) const override;
Function *getCalledScalarFunction() const {
- return cast<Function>(
- getOperand(getNumOperands() - 1)->getLiveInIRValue());
+ return cast<Function>(getOperand(getNumOperands() - 1)->getLiveInIRValue());
}
operand_range args() { return drop_end(operands()); }
diff --git a/llvm/lib/Transforms/Vectorize/VPlanRecipes.cpp b/llvm/lib/Transforms/Vectorize/VPlanRecipes.cpp
index d90c2c00e..c5e171486 100644
--- a/llvm/lib/Transforms/Vectorize/VPlanRecipes.cpp
+++ b/llvm/lib/Transforms/Vectorize/VPlanRecipes.cpp
@@ -395,8 +395,7 @@ VPUnrollPartAccessor<PartOpIdx>::getUnrollPartOperand(const VPUser &U) const {
template <unsigned PartOpIdx>
unsigned VPUnrollPartAccessor<PartOpIdx>::getUnrollPart(const VPUser &U) const {
if (auto *UnrollPartOp = getUnrollPartOperand(U))
- return cast<ConstantInt>(UnrollPartOp->getLiveInIRValue())
- ->getZExtValue();
+ return cast<ConstantInt>(UnrollPartOp->getLiveInIRValue())->getZExtValue();
return 0;
}
@@ -580,8 +579,7 @@ Value *VPInstruction::generate(VPTransformState &State) {
Name);
ElementCount EC = State.VF.multiplyCoefficientBy(
- cast<ConstantInt>(getOperand(2)->getLiveInIRValue())
- ->getZExtValue());
+ cast<ConstantInt>(getOperand(2)->getLiveInIRValue())->getZExtValue());
auto *PredTy = VectorType::get(Builder.getInt1Ty(), EC);
return Builder.CreateIntrinsic(Intrinsic::get_active_lane_mask,
{PredTy, ScalarTC->getType()},
@@ -698,8 +696,7 @@ Value *VPInstruction::generate(VPTransformState &State) {
// If this start vector is scaled then it should produce a vector with fewer
// elements than the VF.
ElementCount VF = State.VF.divideCoefficientBy(
- cast<ConstantInt>(getOperand(2)->getLiveInIRValue())
- ->getZExtValue());
+ cast<ConstantInt>(getOperand(2)->getLiveInIRValue())->getZExtValue());
auto *Iden = Builder.CreateVectorSplat(VF, State.get(getOperand(1), true));
return Builder.CreateInsertElement(Iden, State.get(getOperand(0), true),
Builder.getInt32(0));
@@ -1080,8 +1077,7 @@ InstructionCost VPInstruction::computeCost(ElementCount VF,
case VPInstruction::ActiveLaneMask: {
Type *ArgTy = Ctx.Types.inferScalarType(getOperand(0));
unsigned Multiplier =
- cast<ConstantInt>(getOperand(2)->getLiveInIRValue())
- ->getZExtValue();
+ cast<ConstantInt>(getOperand(2)->getLiveInIRValue())->getZExtValue();
Type *RetTy = toVectorTy(Type::getInt1Ty(Ctx.LLVMCtx), VF * Multiplier);
IntrinsicCostAttributes Attrs(Intrinsic::get_active_lane_mask, RetTy,
{ArgTy, ArgTy});
@@ -3213,8 +3209,8 @@ InstructionCost VPReplicateRecipe::computeCost(ElementCount VF,
// instruction cost.
return 0;
case Instruction::Call: {
- auto *CalledFn = cast<Function>(
- getOperand(getNumOperands() - 1)->getLiveInIRValue());
+ auto *CalledFn =
+ cast<Function>(getOperand(getNumOperands() - 1)->getLiveInIRValue());
SmallVector<const VPValue *> ArgOps(drop_end(operands()));
SmallVector<Type *, 4> Tys;
``````````
</details>
https://github.com/llvm/llvm-project/pull/172758
More information about the llvm-commits
mailing list