Address comments

Author: Sterling-Augustine

llvm/include/llvm/Transforms/Vectorize/SandboxVectorizer/SeedCollector.h

@@ -225,9 +225,8 @@ class SeedContainer {
     /// instructions.
 
     // TODO: Range_size counts fully used-bundles. Further, iterating over
-    // anything other than the Bundles in a SeedContainer includes used seeds,
-    // so for now just check that removing all the seeds from a bundle also
-    // empties the bundle. Rework the iterator logic to clean this up.
+    // anything other than the Bundles in a SeedContainer includes used
+    // seeds. Rework the iterator logic to clean this up.
     iterator(BundleMapT &Map, BundleMapT::iterator MapIt, ValT *Vec, int VecIdx)
         : Map(&Map), MapIt(MapIt), Vec(Vec), VecIdx(VecIdx) {}
     value_type &operator*() {

llvm/unittests/Transforms/Vectorize/SandboxVectorizer/SeedCollectorTest.cpp

@@ -374,11 +374,12 @@ define void @foo(ptr noalias %ptr, float %val) {
   // Expect just one vector of store seeds
   EXPECT_EQ(range_size(StoreSeedsRange), 1u);
   ExpectThatElementsAre(SB, {St0, St1, St2, St3});
-  // Check that the EraseInstr callback works.  TODO: Range_size counts fully
-  // used-bundles even though the iterator skips them. Further, iterating over
-  // anything other than the Bundles in a SeedContainer includes used seeds.  So
-  // for now just check that removing all the seeds from a bundle also empties
-  // the bundle.
+  // Check that the EraseInstr callback works.
+
+  // TODO: Range_size counts fully used-bundles even though the iterator skips
+  // them. Further, iterating over anything other than the Bundles in a
+  // SeedContainer includes used seeds. So for now just check that removing all
+  // the seeds from a bundle also empties the bundle.
   St0->eraseFromParent();
   St1->eraseFromParent();
   St2->eraseFromParent();