[PATCH] D96791: [SLP] Double UserCost compensation for vector store of aggregate

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Mar 13 04:28:07 PST 2021


RKSimon added a comment.

@ABataev Any comments?



================
Comment at: llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp:6298
+        for (User *U: InsertUses[I + OpsWidth - 1]->users())
+          if (isa<StoreInst>(U)) {
+              UserCost *= 2;
----------------
Can we use an llvm::any_of pattern instead here?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D96791/new/

https://reviews.llvm.org/D96791



More information about the llvm-commits mailing list