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

[Justin Lebar via llvm-commits]

jlebar accepted this revision.
This revision is now accepted and ready to land.

================
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;
 }
----------------
asbirlea wrote:
> I think you meant return false on the first condition. Updated.
Yes, "not misaligned".  Sigh.


https://reviews.llvm.org/D22936