Fix typos and format issues

Author: RonDahan101

llvm/lib/Transforms/Vectorize/LoopVectorizationLegality.cpp

@@ -1077,7 +1077,7 @@ bool LoopVectorizationLegality::canVectorizeInstrs() {
 /// When updateOperator can be add, sub, add.sat, umin, umax.
 ///
 /// It matches a pattern starting from \p HSt, which Stores to the 'buckets'
-/// array the computed histogram. It uses a update instruction to update all
+/// array the computed histogram. It uses an update instruction to update all
 /// counts, storing them using a loop-variant index Load from the 'indices'
 /// input array.
 ///
@@ -1099,14 +1099,16 @@ static bool findHistogram(LoadInst *LI, StoreInst *HSt, Loop *TheLoop,
   // FIXME: We assume the loop invariant term is on the RHS.
   //        Fine for an immediate/constant, but maybe not a generic value?
   Value *HIncVal = nullptr;
-  if (!match(HUpdateOp, m_Add(m_Load(m_Specific(HPtrInstr)), m_Value(HIncVal))) &&
-      !match(HUpdateOp, m_Sub(m_Load(m_Specific(HPtrInstr)), m_Value(HIncVal))) &&
+  if (!match(HUpdateOp,
+             m_Add(m_Load(m_Specific(HPtrInstr)), m_Value(HIncVal))) &&
+      !match(HUpdateOp,
+             m_Sub(m_Load(m_Specific(HPtrInstr)), m_Value(HIncVal))) &&
       !match(HUpdateOp, m_Intrinsic<Intrinsic::uadd_sat>(
-                         m_Load(m_Specific(HPtrInstr)), m_Value(HIncVal))) &&
-      !match(HUpdateOp, m_Intrinsic<Intrinsic::umax>(m_Load(m_Specific(HPtrInstr)),
-                                                  m_Value(HIncVal))) &&
-      !match(HUpdateOp, m_Intrinsic<Intrinsic::umin>(m_Load(m_Specific(HPtrInstr)),
-                                                  m_Value(HIncVal))))
+                            m_Load(m_Specific(HPtrInstr)), m_Value(HIncVal))) &&
+      !match(HUpdateOp, m_Intrinsic<Intrinsic::umax>(
+                            m_Load(m_Specific(HPtrInstr)), m_Value(HIncVal))) &&
+      !match(HUpdateOp, m_Intrinsic<Intrinsic::umin>(
+                            m_Load(m_Specific(HPtrInstr)), m_Value(HIncVal))))
     return false;
 
   // Make sure the increment value is loop invariant.

llvm/lib/Transforms/Vectorize/LoopVectorize.cpp

@@ -8657,7 +8657,7 @@ VPRecipeBuilder::tryToWidenHistogram(const HistogramInfo *HI,
   // FIXME: Support other operations.
   unsigned Opcode = HI->Update->getOpcode();
   assert(VPHistogramRecipe::isLegalUpdateInstruction(HI->Update) &&
-         "Found Ilegal update instruction for histogram");
+         "Found Illegal update instruction for histogram");
 
   SmallVector<VPValue *, 3> HGramOps;
   // Bucket address.

llvm/lib/Transforms/Vectorize/VPlanRecipes.cpp

@@ -1238,7 +1238,7 @@ void VPHistogramRecipe::execute(VPTransformState &State) {
   if (Opcode == Instruction::Sub)
     IncAmt = Builder.CreateNeg(IncAmt);
   assert(isLegalUpdateInstruction(UpdateInst) &&
-         "Found Ilegal update instruction for histogram");
+         "Found Illegal update instruction for histogram");
 
   State.Builder.CreateIntrinsic(getHistogramOpcode(UpdateInst),
                                 {VTy, IncAmt->getType()},
@@ -1293,7 +1293,7 @@ void VPHistogramRecipe::print(raw_ostream &O, const Twine &Indent,
                               VPSlotTracker &SlotTracker) const {
   auto *UpdateInst = cast<Instruction>(getOperand(2)->getUnderlyingValue());
   assert(isLegalUpdateInstruction(UpdateInst) &&
-         "Found Ilegal update instruction for histogram");
+         "Found Illegal update instruction for histogram");
   O << Indent << "WIDEN-HISTOGRAM buckets: ";
   getOperand(0)->printAsOperand(O, SlotTracker);
 
@@ -1315,7 +1315,7 @@ void VPHistogramRecipe::print(raw_ostream &O, const Twine &Indent,
       O << ", min: ";
       break;
     default:
-      llvm_unreachable("Found Ilegal update instruction for histogram");
+      llvm_unreachable("Found Illegal update instruction for histogram");
     }
   }
   getOperand(1)->printAsOperand(O, SlotTracker);
@@ -1349,7 +1349,7 @@ unsigned VPHistogramRecipe::getHistogramOpcode(Instruction *I) {
   // We only support add and sub instructions and the following list of
   // intrinsics: uadd.sat, umax, umin.
   assert(isLegalUpdateInstruction(I) &&
-         "Found Ilegal update instruction for histogram");
+         "Found Illegal update instruction for histogram");
   if (isa<BinaryOperator>(I))
     return Intrinsic::experimental_vector_histogram_add;
   auto *II = cast<IntrinsicInst>(I);
@@ -1361,7 +1361,7 @@ unsigned VPHistogramRecipe::getHistogramOpcode(Instruction *I) {
   case Intrinsic::umin:
     return Intrinsic::experimental_vector_histogram_umin;
   default:
-    llvm_unreachable("Found Ilegal update instruction for histogram");
+    llvm_unreachable("Found Illegal update instruction for histogram");
   }
 }