[PATCH] D31394: [X86][Clang] Converting __mm{|256|512}_movm_epi{8|16|32|64} LLVMIR call into generic intrinsics.

[Craig Topper via Phabricator via cfe-commits]

craig.topper added inline comments.


================
Comment at: lib/CodeGen/CGBuiltin.cpp:7176
+  return CGF.Builder.CreateSExt(
+      Mask, llvm::VectorType::get(IT, NumberOfElements), "vpmovm2");
+}
----------------
m_zuckerman wrote:
> craig.topper wrote:
> > Why can't we just use DstTy here to create the SExt?
> You are right Thanks
That makes the IT parameter unnecessary now right? Which means we don't need to pass different values at the call site below.


https://reviews.llvm.org/D31394