[llvm] r244916 - [ARM] FMINNAN/FMAXNAN of f64 are not legal.

[James Molloy via llvm-commits]

Author: jamesm
Date: Thu Aug 13 12:28:26 2015
New Revision: 244916

URL: http://llvm.org/viewvc/llvm-project?rev=244916&view=rev
Log:
[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.

Modified:
    llvm/trunk/lib/Target/ARM/ARMISelLowering.cpp

Modified: llvm/trunk/lib/Target/ARM/ARMISelLowering.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/ARM/ARMISelLowering.cpp?rev=244916&r1=244915&r2=244916&view=diff
==============================================================================
--- llvm/trunk/lib/Target/ARM/ARMISelLowering.cpp (original)
+++ llvm/trunk/lib/Target/ARM/ARMISelLowering.cpp Thu Aug 13 12:28:26 2015
@@ -954,8 +954,6 @@ ARMTargetLowering::ARMTargetLowering(con
   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);