[PATCH] D22534: [LSV] Insert stores at the right point.

Alina Sbirlea via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 19 15:23:55 PDT 2016


asbirlea added inline comments.

================
Comment at: lib/Transforms/Vectorize/LoadStoreVectorizer.cpp:772
@@ -777,1 +771,3 @@
+  BasicBlock::iterator First, Last;
+  std::tie(First, Last) = getBoundaryInstrs(Chain);
   Builder.SetInsertPoint(&*Last);
----------------
Can you add a FIXME to mark the possible performance penalty for calling this again?
Not a concern right now, but it would be good to keep a record of it.


https://reviews.llvm.org/D22534





More information about the llvm-commits mailing list