[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