[PATCH] D17485: [x86, InstCombine] transform x86 AVX masked stores to LLVM intrinsics

Sanjay Patel via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 25 08:11:36 PST 2016


spatel marked an inline comment as done.

================
Comment at: lib/Transforms/InstCombine/InstCombineCalls.cpp:855
@@ +854,3 @@
+  // the LLVM intrinsic definition for the pointer argument.
+  unsigned AddrSpace = cast<PointerType>(Ptr->getType())->getAddressSpace();
+  PointerType *VecPtrTy = PointerType::get(Vec->getType(), AddrSpace);
----------------
Thanks - I didn't think it was possible to specify the address space with the AVX intrinsic, but we should do the right thing here anyway.


http://reviews.llvm.org/D17485





More information about the llvm-commits mailing list