Address review

Author: SamTebbs33

llvm/lib/Transforms/Vectorize/LoopVectorize.cpp

@@ -7608,7 +7608,7 @@ static bool planContainsAdditionalSimplifications(VPlan &Plan,
       }
       // The VPlan-based cost model is more accurate for partial reduction and
       // comparing against the legacy cost isn't desirable.
-      if (dyn_cast<VPPartialReductionRecipe>(&R))
+      if (isa<VPPartialReductionRecipe>(&R))
         return true;
       if (Instruction *UI = GetInstructionForCost(&R))
         SeenInstrs.insert(UI);

llvm/lib/Transforms/Vectorize/VPlan.h

@@ -2447,6 +2447,7 @@ class VPPartialReductionRecipe : public VPSingleDefRecipe {
            "Unexpected operand order for partial reduction recipe");
   }
   ~VPPartialReductionRecipe() override = default;
+
   VPPartialReductionRecipe *clone() override {
     return new VPPartialReductionRecipe(Opcode, getOperand(0), getOperand(1));
   }

llvm/lib/Transforms/Vectorize/VPlanRecipes.cpp

@@ -337,7 +337,7 @@ void VPPartialReductionRecipe::execute(VPTransformState &State) {
 
   CallInst *V = Builder.CreateIntrinsic(
       RetTy, Intrinsic::experimental_vector_partial_reduce_add,
-      {PhiVal, BinOpVal}, nullptr, Twine("partial.reduce"));
+      {PhiVal, BinOpVal}, nullptr, "partial.reduce");
 
   State.set(this, V);
 }