[PATCH] D34601: [X86][LLVM]Expanding Supports lowerInterleavedStore() in X86InterleavedAccess.

michael zuckerman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 28 08:38:38 PDT 2017


m_zuckerman added a comment.

In https://reviews.llvm.org/D34601#791846, @dorit wrote:

> Have you tried enabling this also for AVX? (I understand if not, because with the current cost numbers that TTI returns for interleaved accesses on AVX we'll probably determine it's not worth vectorizing... so that may need to come along with an update of getInterleavedMemoryOpCost -- maybe at least a TODO comment is needed).


Thanks , I tried it and the result was good for AVX. 
You can see in the test **interleavedStore.all** that we are getting less instructions compared to the old code.


https://reviews.llvm.org/D34601





More information about the llvm-commits mailing list