[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 cfe-commits at lists.llvm.org
Mon Apr 3 12:35:41 PDT 2017


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





More information about the cfe-commits mailing list