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

Matt Arsenault via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 28 14:20:08 PDT 2016


arsenm added inline comments.

================
Comment at: lib/Transforms/Vectorize/LoadStoreVectorizer.cpp:43
@@ -42,5 +42,3 @@
 
-// TODO: Remove this
-static const unsigned TargetBaseAlign = 4;
-
+static const unsigned StackAdjustedAlignment = 4;
 typedef SmallVector<Instruction *, 8> InstrList;
----------------
jlebar wrote:
> I think we still want a comment here, because we're basically assuming that 4 bytes is good enough for any type.
I think this makes an easier TTI hook now that it isn't related to unaligned access


https://reviews.llvm.org/D22936





More information about the llvm-commits mailing list