[PATCH] D21458: LoadStoreVectorizer: Fix crashes on sub-byte types

[Matt Arsenault via llvm-commits]

arsenm closed this revision.
arsenm added a comment.

r274306


================
Comment at: lib/Transforms/Vectorize/LoadStoreVectorizer.cpp:471
@@ +470,3 @@
+      if (TySize < 8)
+        continue;
+
----------------
jlebar wrote:
> Just checking, there are sub-byte types, but size >= 8 implies size % 8 == 0?
I think the % 8 is taken care of by the weird type alloc size check, so this might be redundant and could use simplification


http://reviews.llvm.org/D21458