[PATCH] D12985: [ARM] Take into account address spaces in interleaved access vectorization

silviu.baranga@arm.com via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 21 02:40:39 PDT 2015


sbaranga added inline comments.

================
Comment at: lib/Target/ARM/ARMISelLowering.cpp:11708
@@ +11707,3 @@
+  Type *Int8Ptr = Builder.getInt8PtrTy();
+  Ops.push_back(Builder.CreateAddrSpaceCast(
+      Builder.CreateBitCast(LI->getPointerOperand(), Int8PtrAddr), Int8Ptr));
----------------
This is interesting.. We ideally wouldn't want to introduce address space casts when not necessary (I can imagine some users using the address space qualifier to help with alias analysis).

We should be able to get a variant of the vldn intrinsic that operates on the correct address space.


http://reviews.llvm.org/D12985





More information about the llvm-commits mailing list