[PATCH] Address Space Casting
Michele Scandale
michele.scandale at gmail.com
Mon Nov 11 02:45:54 PST 2013
================
Comment at: lib/IR/AutoUpgrade.cpp:458-461
@@ +457,6 @@
+ Type *SrcTy = V->getType();
+ if (SrcTy->isPointerTy() && DestTy->isPointerTy() &&
+ SrcTy->getPointerAddressSpace() != DestTy->getPointerAddressSpace()) {
+ LLVMContext &Context = V->getContext();
+
+ // We have no information about target data layout, so we assume that
----------------
Matt Arsenault wrote:
> This won't work for vectors of pointers
I haven't considered vector of pointers as supported types: such test is used also in other places of this patch.
Do you think this case (e.g. <4 x i32*> --> <4 x i32 addrspace(1)*>) should be handled by addrspacecast instruction? In this case I'll update the patch asap.
http://llvm-reviews.chandlerc.com/D1401
More information about the llvm-commits
mailing list