From 1da1d0218a4a98ca639cbf0555768342e6356d78 Mon Sep 17 00:00:00 2001 From: James Molloy Date: Thu, 13 Aug 2015 17:28:26 +0000 Subject: [PATCH] [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. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@244916 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/Target/ARM/ARMISelLowering.cpp | 2 -- 1 file changed, 2 deletions(-) diff --git a/lib/Target/ARM/ARMISelLowering.cpp b/lib/Target/ARM/ARMISelLowering.cpp index c0eb45ca043..a628b22106b 100644 --- a/lib/Target/ARM/ARMISelLowering.cpp +++ b/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); -- 2.34.1