diff options
author | Sam Parker <sam.parker@arm.com> | 2017-09-27 08:30:45 +0000 |
---|---|---|
committer | Sam Parker <sam.parker@arm.com> | 2017-09-27 08:30:45 +0000 |
commit | 211f47aa37ce69f89adf62c772768bc2a33696ae (patch) | |
tree | e8034111c0e94c5111fee4e2b3b4129d020fae16 | |
parent | caf6fd51eee1eb2103b25ad006dc12af8606eb70 (diff) | |
download | bcm5719-llvm-211f47aa37ce69f89adf62c772768bc2a33696ae.tar.gz bcm5719-llvm-211f47aa37ce69f89adf62c772768bc2a33696ae.zip |
[ARM] isTruncateFree fix
I implemented isTruncateFree in rL313533, this patch fixes the logic
to match my comment, as the previous logic was too general. Now the
only truncates that are free are i64 -> i32.
Differential Revision: https://reviews.llvm.org/D38234
llvm-svn: 314280
-rw-r--r-- | llvm/lib/Target/ARM/ARMISelLowering.cpp | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/llvm/lib/Target/ARM/ARMISelLowering.cpp b/llvm/lib/Target/ARM/ARMISelLowering.cpp index e3198a4b8fb..333a077e640 100644 --- a/llvm/lib/Target/ARM/ARMISelLowering.cpp +++ b/llvm/lib/Target/ARM/ARMISelLowering.cpp @@ -12320,18 +12320,18 @@ EVT ARMTargetLowering::getOptimalMemOpType(uint64_t Size, bool ARMTargetLowering::isTruncateFree(Type *SrcTy, Type *DstTy) const { if (!SrcTy->isIntegerTy() || !DstTy->isIntegerTy()) return false; - unsigned NumBits1 = SrcTy->getPrimitiveSizeInBits(); - unsigned NumBits2 = DstTy->getPrimitiveSizeInBits(); - return NumBits1 > NumBits2; + unsigned SrcBits = SrcTy->getPrimitiveSizeInBits(); + unsigned DestBits = DstTy->getPrimitiveSizeInBits(); + return (SrcBits == 64 && DestBits == 32); } bool ARMTargetLowering::isTruncateFree(EVT SrcVT, EVT DstVT) const { if (SrcVT.isVector() || DstVT.isVector() || !SrcVT.isInteger() || !DstVT.isInteger()) return false; - unsigned NumBits1 = SrcVT.getSizeInBits(); - unsigned NumBits2 = DstVT.getSizeInBits(); - return NumBits1 > NumBits2; + unsigned SrcBits = SrcVT.getSizeInBits(); + unsigned DestBits = DstVT.getSizeInBits(); + return (SrcBits == 64 && DestBits == 32); } bool ARMTargetLowering::isZExtFree(SDValue Val, EVT VT2) const { |