> Oh, I get it now.  No, there is no reason for the duplication between
> minmax_operator and the insn itself.  You should be able to remove the
> tests from minmax_operator.  I wonder why they are there at all?

This, then?

2007-03-15  DJ Delorie  <[EMAIL PROTECTED]>

        * config/frv/predicates.md (minmax_operator): Don't check operands
        here.

Index: config/frv/predicates.md
===================================================================
--- config/frv/predicates.md    (revision 122960)
+++ config/frv/predicates.md    (working copy)
@@ -1271,12 +1271,6 @@
       break;
     }
 
-  if (! integer_register_operand (XEXP (op, 0), mode))
-    return FALSE;
-
-  if (! gpr_or_int10_operand (XEXP (op, 1), mode))
-    return FALSE;
-
   return TRUE;
 })
 

Reply via email to