[PATCH] D24681: Optimize patterns of vectorized interleaved memory accesses for X86.

Elena Demikhovsky via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 7 07:39:12 PDT 2016


delena added a comment.

What about shuffles set for AVX-512?



================
Comment at: lib/Target/X86/X86InterleavedAccess.cpp:143
+
+  return lower(LI, Shuffles, Indices, Factor);
+}
----------------
Farhana wrote:
> delena wrote:
> > It is not a good name for function. I think that you don't need additional function call here at all.
> You are right in the current context. My plan is to define a class to encapsulate all the information and allow data sharing where I will have two main functions one for load generation and the other one for shuffle-generation. In order to keep the follow-up change-set with minimal changes I decided to create a function here. I hope it's ok to do so.
Each patch should look good regardless of future plans.  


https://reviews.llvm.org/D24681





More information about the llvm-commits mailing list