!fixup address latest comments, thanks!

Author: fhahn

llvm/lib/Transforms/Vectorize/VPlan.h

@@ -1961,8 +1961,8 @@ class VPScalarPHIRecipe : public VPHeaderPHIRecipe {
 /// A recipe for widened phis. Incoming values are operands of the recipe and
 /// their operand index corresponds to the incoming predeocessor block. If the
 /// recipe is placed in an entry block to a (non-replicate) region, it must have
-/// exactly 2 incoming values, the first from the predecessors of the region and the second
-/// from the exiting block of the region.
+/// exactly 2 incoming values, the first from the predecessors of the region and
+/// the second from the exiting block of the region.
 class VPWidenPHIRecipe : public VPSingleDefRecipe {
 public:
   /// Create a new VPWidenPHIRecipe for \p Phi with start value \p Start and

llvm/lib/Transforms/Vectorize/VPlanHCFGBuilder.cpp

@@ -151,7 +151,8 @@ void PlainCFGBuilder::fixPhiNodes() {
           getOrCreateVPOperand(Phi->getIncomingValue(I));
     }
     for (VPBlockBase *Pred : VPPhi->getParent()->getPredecessors())
-      VPPhi->addOperand(VPPredToIncomingValue.lookup(Pred->getExitingBasicBlock()));
+      VPPhi->addOperand(
+          VPPredToIncomingValue.lookup(Pred->getExitingBasicBlock()));
   }
 }
 

llvm/lib/Transforms/Vectorize/VPlanRecipes.cpp

@@ -3614,10 +3614,14 @@ VPBasicBlock *VPWidenPHIRecipe::getIncomingBlock(unsigned I) {
   VPBlockBase *Pred = nullptr;
   if (Parent->getNumPredecessors() == 0) {
     auto *Region = Parent->getParent();
-    assert(Region && !Region->isReplicator() && Region->getEntry() == Parent && "must be in the entry block of a non-replicate region");
-    assert(I < 2 && getNumOperands() == 2 && "when placed in an entry block, only 2 incoming blocks are available");
+    assert(Region && !Region->isReplicator() && Region->getEntry() == Parent &&
+           "must be in the entry block of a non-replicate region");
+    assert(
+        I < 2 && getNumOperands() == 2 &&
+        "when placed in an entry block, only 2 incoming blocks are available");
 
-    // I ==  0 selects the predecessor of the region, I == 1 selects the region itself whose exiting block feeds the phi across the backedge.
+    // I ==  0 selects the predecessor of the region, I == 1 selects the region
+    // itself whose exiting block feeds the phi across the backedge.
     Pred = I == 0 ? Region->getSinglePredecessor() : Region;
   } else {
     Pred = Parent->getPredecessors()[I];