> 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; })