[PATCH] D22936: Remove TargetBaseAlign. Keep alignment for stack adjustments.

[Alina Sbirlea via llvm-commits]

asbirlea added inline comments.

================
Comment at: lib/Transforms/Vectorize/LoadStoreVectorizer.cpp:1036
@@ -1034,5 +1035,3 @@
                                                    Alignment, &Fast);
-  // TODO: Remove TargetBaseAlign
-  return !(Allows && Fast) && (Alignment % SzInBytes) != 0 &&
-         (Alignment % TargetBaseAlign) != 0;
+  return !Allows || !Fast;
 }
----------------
I think you meant return false on the first condition. Updated.


https://reviews.llvm.org/D22936