[PATCH] D23906: LSV: Fix incorrectly increasing alignment

Matt Arsenault via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 8 21:09:40 PDT 2016


arsenm added inline comments.

================
Comment at: lib/Transforms/Vectorize/LoadStoreVectorizer.cpp:798
@@ -800,1 +797,3 @@
+                                                   DL, S0, nullptr, &DT);
+    if (NewAlign < StackAdjustedAlignment)
       return false;
----------------
It doesn't actually return 0 on failure, it returns the known alignment. In the motivating testcase, load_unknown_offset_align1_i8, it returns 1 when it can't find the constant offset


https://reviews.llvm.org/D23906





More information about the llvm-commits mailing list