diff options
| author | Nemanja Ivanovic <nemanja.i.ibm@gmail.com> | 2020-04-20 12:59:47 -0500 |
|---|---|---|
| committer | Tom Stellard <tstellar@redhat.com> | 2020-05-19 08:31:01 -0700 |
| commit | 3de89abf189e63da4edbfda509ed3d421bdba1a9 (patch) | |
| tree | 05c1f39d25202c14d1a151f2127ad6a4bac8bb06 /llvm/lib/Target | |
| parent | 17d0ddd522bfed254f1a6314996d678686522309 (diff) | |
| download | bcm5719-llvm-3de89abf189e63da4edbfda509ed3d421bdba1a9.tar.gz bcm5719-llvm-3de89abf189e63da4edbfda509ed3d421bdba1a9.zip | |
[PowerPC] Do not attempt to reuse load for 64-bit FP_TO_UINT without FPCVT
We call the function that attempts to reuse the conversion without checking
whether the target matches the constraints that the callee expects. This patch
adds the check prior to the call.
Fixes: https://bugs.llvm.org/show_bug.cgi?id=43976
Differential revision: https://reviews.llvm.org/D77564
(cherry picked from commit 64b31d96dfd6c05e6d52d8798726dec60502cfde)
Diffstat (limited to 'llvm/lib/Target')
| -rw-r--r-- | llvm/lib/Target/PowerPC/PPCISelLowering.cpp | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/llvm/lib/Target/PowerPC/PPCISelLowering.cpp b/llvm/lib/Target/PowerPC/PPCISelLowering.cpp index 352a05529bc..00f59bba52e 100644 --- a/llvm/lib/Target/PowerPC/PPCISelLowering.cpp +++ b/llvm/lib/Target/PowerPC/PPCISelLowering.cpp @@ -7851,9 +7851,10 @@ bool PPCTargetLowering::canReuseLoadAddress(SDValue Op, EVT MemVT, SelectionDAG &DAG, ISD::LoadExtType ET) const { SDLoc dl(Op); + bool ValidFPToUint = Op.getOpcode() == ISD::FP_TO_UINT && + (Subtarget.hasFPCVT() || Op.getValueType() == MVT::i32); if (ET == ISD::NON_EXTLOAD && - (Op.getOpcode() == ISD::FP_TO_UINT || - Op.getOpcode() == ISD::FP_TO_SINT) && + (ValidFPToUint || Op.getOpcode() == ISD::FP_TO_SINT) && isOperationLegalOrCustom(Op.getOpcode(), Op.getOperand(0).getValueType())) { |
