From 31117875c2888a8b341829bde1812b3989af1df1 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. llvm-svn: 244916 --- llvm/lib/Target/ARM/ARMISelLowering.cpp | 2 -- 1 file changed, 2 deletions(-) diff --git a/llvm/lib/Target/ARM/ARMISelLowering.cpp b/llvm/lib/Target/ARM/ARMISelLowering.cpp index c0eb45c..a628b22 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); -- 2.7.4