fix the comments

Author: LiqinWeng

llvm/lib/Transforms/Vectorize/VPlanRecipes.cpp

@@ -960,14 +960,18 @@ void VPWidenIntrinsicRecipe::execute(VPTransformState &State) {
     Args.push_back(Arg);
   }
 
-  if (VPIntrinsic::isVPIntrinsic(VectorIntrinsicID) &&
-      VectorIntrinsicID != Intrinsic::vp_select) {
-    Value *Mask =
-        State.Builder.CreateVectorSplat(State.VF, State.Builder.getTrue());
+  if (VPIntrinsic::isVPIntrinsic(VectorIntrinsicID)) {
     Value *EVL = Args.back();
     Args.pop_back();
-    Args.push_back(Mask);
-    Args.push_back(EVL);
+    // Add EVL && Mask Ops for vector-predication intrinsics.
+    if (VPIntrinsic::getMaskParamPos(VectorIntrinsicID)) {
+      Value *Mask =
+          State.Builder.CreateVectorSplat(State.VF, State.Builder.getTrue());
+      Args.push_back(Mask);
+    }
+    if (VPIntrinsic::getVectorLengthParamPos(VectorIntrinsicID)) {
+      Args.push_back(EVL);
+    }
   }
 
   // Use vector version of the intrinsic.
@@ -982,6 +986,7 @@ void VPWidenIntrinsicRecipe::execute(VPTransformState &State) {
     CI->getOperandBundlesAsDefs(OpBundles);
 
   CallInst *V = State.Builder.CreateCall(VectorF, Args, OpBundles);
+
   setFlags(V);
 
   if (!V->getType()->isVoidTy())

llvm/lib/Transforms/Vectorize/VPlanTransforms.cpp

@@ -1484,13 +1484,13 @@ static void transformRecipestoEVLRecipes(VPlan &Plan, VPValue &EVL) {
               .Case<VPWidenIntrinsicRecipe>(
                   [&](VPWidenIntrinsicRecipe *CInst) -> VPRecipeBase * {
                     auto *CI = cast<CallInst>(CInst->getUnderlyingInstr());
-                    SmallVector<VPValue *> Ops(CInst->operands());
-                    Ops.push_back(&EVL);
-
                     Intrinsic::ID VPID = VPIntrinsic::getForIntrinsic(
                         CI->getCalledFunction()->getIntrinsicID());
                     if (VPID == Intrinsic::not_intrinsic)
                       return nullptr;
+
+                    SmallVector<VPValue *> Ops(CInst->operands());
+                    Ops.push_back(&EVL);
                     return new VPWidenIntrinsicRecipe(
                         *CI, VPID, Ops, CI->getType(), CI->getDebugLoc());
                   })

llvm/test/Transforms/LoopVectorize/RISCV/vplan-vp-call-intrinsics.ll

@@ -474,4 +474,4 @@ declare i32 @llvm.ctlz.i32(i32, i1 immarg)
 declare i32 @llvm.cttz.i32(i32, i1 immarg)
 declare i64 @llvm.lrint.i64.f64(double)
 declare i64 @llvm.llrint.i64.f64(double)
-declare i32 @llvm.abs.i32(i32, i1 immarg)
+declare i32 @llvm.abs.i32(i32, i1 immarg)