[llvm] [VPlan] Run more passes through `RUN_VPLAN_PASS[_NO_VERIFY]` (PR #180580)

[Andrei Elovikov via llvm-commits]

================
@@ -7429,9 +7429,10 @@ DenseMap<const SCEV *, Value *> LoopVectorizationPlanner::executePlan(
   // Retrieving VectorPH now when it's easier while VPlan still has Regions.
   VPBasicBlock *VectorPH = cast<VPBasicBlock>(BestVPlan.getVectorPreheader());
 
-  VPlanTransforms::optimizeForVFAndUF(BestVPlan, BestVF, BestUF, PSE);
-  VPlanTransforms::simplifyRecipes(BestVPlan);
-  VPlanTransforms::removeBranchOnConst(BestVPlan);
+  RUN_VPLAN_PASS_NO_VERIFY(VPlanTransforms::optimizeForVFAndUF, BestVPlan,
----------------
eas wrote:

>From https://llvm.org/docs/CodingStandards.html#name-types-functions-variables-and-enumerators-properly:

> Function names should be verb phrases (as they represent actions), and command-like function should be imperative.

and this macro acts "as a function" (or would have been one if we had reflection capabilities to get a string representing the function parameter), so I'm very hesitant about dropping the "RUN" part.

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