[PATCH] SSE pslldq/psrldq shuffle mask decodes
Chandler Carruth
chandlerc at gmail.com
Sun Oct 5 16:25:27 PDT 2014
================
Comment at: lib/Target/X86/Utils/X86ShuffleDecode.cpp:83-84
@@ +82,4 @@
+void DecodePSLLDQMask(MVT VT, unsigned Imm, SmallVectorImpl<int> &ShuffleMask) {
+ unsigned NumElts = VT.getVectorNumElements();
+ unsigned Offset = Imm * (VT.getVectorElementType().getSizeInBits() / 8);
+
----------------
I wouldn't decode this in terms of the vector element type. The mask should be a byte-vector mask in all cases, and VT should just provide the size of the vector (128-bit, 256-bit, 512-bit, whatever).
http://reviews.llvm.org/D5598
More information about the llvm-commits
mailing list