diff options
author | Sanjoy Das <sanjoy@playingwithpointers.com> | 2017-06-17 20:28:13 +0000 |
---|---|---|
committer | Sanjoy Das <sanjoy@playingwithpointers.com> | 2017-06-17 20:28:13 +0000 |
commit | b70ddd8901c0844bf0f41bd1e5326264c30065a3 (patch) | |
tree | 26e9835a18f48d2bb01d0a80c709e87b2a7e2c83 /llvm/lib/Transforms/Scalar/SROA.cpp | |
parent | 91d9bfcec59e8731e14b63ca81d4832fea2e47e4 (diff) | |
download | bcm5719-llvm-b70ddd8901c0844bf0f41bd1e5326264c30065a3.tar.gz bcm5719-llvm-b70ddd8901c0844bf0f41bd1e5326264c30065a3.zip |
[SROA] Add support for non-integral pointers
Summary: C.f. http://llvm.org/docs/LangRef.html#non-integral-pointer-type
Reviewers: chandlerc, loladiro
Reviewed By: loladiro
Subscribers: reames, loladiro, mcrosier, llvm-commits
Differential Revision: https://reviews.llvm.org/D32203
llvm-svn: 305639
Diffstat (limited to 'llvm/lib/Transforms/Scalar/SROA.cpp')
-rw-r--r-- | llvm/lib/Transforms/Scalar/SROA.cpp | 13 |
1 files changed, 11 insertions, 2 deletions
diff --git a/llvm/lib/Transforms/Scalar/SROA.cpp b/llvm/lib/Transforms/Scalar/SROA.cpp index 1527f15f18a..f32d2d58229 100644 --- a/llvm/lib/Transforms/Scalar/SROA.cpp +++ b/llvm/lib/Transforms/Scalar/SROA.cpp @@ -1637,8 +1637,17 @@ static bool canConvertValue(const DataLayout &DL, Type *OldTy, Type *NewTy) { return cast<PointerType>(NewTy)->getPointerAddressSpace() == cast<PointerType>(OldTy)->getPointerAddressSpace(); } - if (NewTy->isIntegerTy() || OldTy->isIntegerTy()) - return true; + + // We can convert integers to integral pointers, but not to non-integral + // pointers. + if (OldTy->isIntegerTy()) + return !DL.isNonIntegralPointerType(NewTy); + + // We can convert integral pointers to integers, but non-integral pointers + // need to remain pointers. + if (!DL.isNonIntegralPointerType(OldTy)) + return NewTy->isIntegerTy(); + return false; } |