[PATCH] D36058: [X86][LLVM]Expanding Supports lowerInterleavedStore() in X86InterleavedAccess (VF8 stride 4).

Farhana Aleen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 11 11:47:10 PDT 2017


Farhana added inline comments.


================
Comment at: lib/Target/X86/X86InterleavedAccess.cpp:124
+  if (ShuffleElemSize == 8 && isa<StoreInst>(Inst) &&
+      (WideInstSize == 256 || WideInstSize == 512 || WideInstSize == 1024))
+    return true;
----------------
Farhana wrote:
> Please remove the extra space after "&&". 
> 
Do we also need to check DL.getTypeSizeInBits(ShuffleVecTy) == 256 here? 


https://reviews.llvm.org/D36058





More information about the llvm-commits mailing list