diff options
author | James Molloy <james.molloy@arm.com> | 2015-08-13 17:28:26 +0000 |
---|---|---|
committer | James Molloy <james.molloy@arm.com> | 2015-08-13 17:28:26 +0000 |
commit | 31117875c2888a8b341829bde1812b3989af1df1 (patch) | |
tree | 7048a30650a9d9a7c778ca0c8269c8dbabcbf728 | |
parent | c71f78f49f6c4a7406abf1402d1e456d9b958198 (diff) | |
download | bcm5719-llvm-31117875c2888a8b341829bde1812b3989af1df1.tar.gz bcm5719-llvm-31117875c2888a8b341829bde1812b3989af1df1.zip |
[ARM] FMINNAN/FMAXNAN of f64 are not legal.
This was my error. We've got f32 marked as legal because they're simulated using a v2f32 instruction, but there's no equivalent for f64.
This will get test coverage imminently when D12015 lands.
llvm-svn: 244916
-rw-r--r-- | llvm/lib/Target/ARM/ARMISelLowering.cpp | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/llvm/lib/Target/ARM/ARMISelLowering.cpp b/llvm/lib/Target/ARM/ARMISelLowering.cpp index c0eb45ca043..a628b22106b 100644 --- a/llvm/lib/Target/ARM/ARMISelLowering.cpp +++ b/llvm/lib/Target/ARM/ARMISelLowering.cpp @@ -954,8 +954,6 @@ ARMTargetLowering::ARMTargetLowering(const TargetMachine &TM, if (Subtarget->hasVFP3()) { setOperationAction(ISD::FMINNAN, MVT::f32, Legal); setOperationAction(ISD::FMAXNAN, MVT::f32, Legal); - setOperationAction(ISD::FMINNAN, MVT::f64, Legal); - setOperationAction(ISD::FMAXNAN, MVT::f64, Legal); } if (Subtarget->hasNEON()) { setOperationAction(ISD::FMINNAN, MVT::v2f32, Legal); |