Address comments

Author: Sterling-Augustine

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

@@ -486,6 +486,7 @@ define void @foo(ptr noalias %ptr, <2 x float> %val0) {
   %r1 = load <2 x float>, ptr %ptr1
   %r2 = load atomic i64, ptr %ptr0 unordered, align 8
   %r3 = load volatile i64, ptr %ptr1
+  %r4 = load void()*, ptr %ptr1
 
   ret void
 }
@@ -504,16 +505,17 @@ define void @foo(ptr noalias %ptr, <2 x float> %val0) {
   auto BB = F.begin();
   sandboxir::SeedCollector SC(&*BB, SE);
 
-  // Find the stores
+  // Find the loads
   auto It = std::next(BB->begin(), 2);
   // StX with X as the order by offset in memory
-  auto *Ld0 = &*It++;
-  auto *Ld1 = &*It++;
+  auto *Ld0 = cast<sandboxir::LoadInst>(&*It++);
+  auto *Ld1 = cast<sandboxir::LoadInst>(&*It++);
 
   auto LoadSeedsRange = SC.getLoadSeeds();
-  EXPECT_EQ(range_size(LoadSeedsRange), 1u);
+  EXPECT_EQ(range_size(LoadSeedsRange), 2u);
   auto &SB = *LoadSeedsRange.begin();
-  // isValidMemSeed check: The atomic and volatile stores should not
-  // be included in the bundle, but the vector stores should be.
+  // isValidMemSeed check: The atomic and volatile loads should not
+  // be included in the bundle, the vector stores should be, but the
+  // void-typed load should not.
   ExpectThatElementsAre(SB, {Ld0, Ld1});
 }