[llvm] [SandboxVectorizer] Register erase callback for seed collection (PR #115951)

via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 12 14:49:40 PST 2024


================
@@ -374,6 +374,21 @@ 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
----------------
vporpo wrote:

nit: TODO: in a separate line. 
also double whitespace before "So".

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


More information about the llvm-commits mailing list