[PATCH] D13277: [SLP] Don't vectorize loads of non-packed types (like i1, i2).

[Michael Zolotukhin via llvm-commits]

mzolotukhin added a comment.

Hi Nadav,

Currently we can't get into such situation with stores, but that's just by luck - the minimal vector size is 128 bit, and maximum number of accesses we can bundle together is 16. So the minimal element size that could be used in vector store is i8, which doesn't have such a problem.

Next, I'm going to reveal the incorrect behavior on stores by replacing these hardcoded parameters with cl options (see e.g. http://reviews.llvm.org/D13278), fix it, and make sure we honor these constraints when vectorizing phis (currently we don't).

Thanks,
Michael


Repository:
  rL LLVM

http://reviews.llvm.org/D13277