[PATCH] D22119: Extended LoadStoreVectorizer to vectorize subchains.

Alina Sbirlea via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 11 07:31:21 PDT 2016


asbirlea added inline comments.

================
Comment at: lib/Transforms/Vectorize/LoadStoreVectorizer.cpp:443
@@ -439,3 +442,3 @@
       DEBUG(dbgs() << "LSV: Found side-effecting operation: " << *I << '\n');
-      return false;
+      return 0;
     }
----------------
jlebar wrote:
> Did you intend to address this comment in your latest update?
No, let's handle that in a separate patch. Yes, it's another feature that should be added.


http://reviews.llvm.org/D22119





More information about the llvm-commits mailing list