[llvm] [SLP] More OOP to simplify vectorizeStores() (NFC) (PR #134605)

Gaƫtan Bossu via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 7 03:37:47 PDT 2025


================
@@ -20694,52 +20735,34 @@ bool SLPVectorizerPass::vectorizeStores(
   // dependencies and no need to waste compile time to try to vectorize them.
   // - Try to vectorize the sequence {1, {1, 0}, {3, 2}}.
----------------
gbossu wrote:

Now that `FillStoresSet()` is more concise and has simpler control flow, I'm wondering if this long comment on top of it could be shortened. What do you think @alexey-bataev?

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


More information about the llvm-commits mailing list