As per the subject. Tested by making sure that there were no new warnings building s390-linux-gnu, and that there were no changes in the assembly output for the C and C++ testsuite. OK to install?
Richard gcc/ * config/s390/s390.md: Use match_test rather than eq/ne symbol_ref throughout file. Index: gcc/config/s390/s390.md =================================================================== --- gcc/config/s390/s390.md 2011-09-13 18:43:39.000000000 +0100 +++ gcc/config/s390/s390.md 2011-09-13 19:02:44.000000000 +0100 @@ -283,31 +283,31 @@ (define_attr "enabled" "" (const_int 1) (and (eq_attr "cpu_facility" "ieee") - (ne (symbol_ref "TARGET_CPU_IEEE_FLOAT") (const_int 0))) + (match_test "TARGET_CPU_IEEE_FLOAT")) (const_int 1) (and (eq_attr "cpu_facility" "zarch") - (ne (symbol_ref "TARGET_ZARCH") (const_int 0))) + (match_test "TARGET_ZARCH")) (const_int 1) (and (eq_attr "cpu_facility" "longdisp") - (ne (symbol_ref "TARGET_LONG_DISPLACEMENT") (const_int 0))) + (match_test "TARGET_LONG_DISPLACEMENT")) (const_int 1) (and (eq_attr "cpu_facility" "extimm") - (ne (symbol_ref "TARGET_EXTIMM") (const_int 0))) + (match_test "TARGET_EXTIMM")) (const_int 1) (and (eq_attr "cpu_facility" "dfp") - (ne (symbol_ref "TARGET_DFP") (const_int 0))) + (match_test "TARGET_DFP")) (const_int 1) (and (eq_attr "cpu_facility" "z10") - (ne (symbol_ref "TARGET_Z10") (const_int 0))) + (match_test "TARGET_Z10")) (const_int 1) (and (eq_attr "cpu_facility" "z196") - (ne (symbol_ref "TARGET_Z196") (const_int 0))) + (match_test "TARGET_Z196")) (const_int 1)] (const_int 0))) @@ -7726,7 +7726,7 @@ (define_insn "*cjump_31" [(set_attr "op_type" "RI") (set_attr "type" "branch") (set (attr "length") - (if_then_else (eq (symbol_ref "flag_pic") (const_int 0)) + (if_then_else (not (match_test "flag_pic")) (if_then_else (lt (abs (minus (pc) (match_dup 0))) (const_int 60000)) (const_int 4) (const_int 6)) (if_then_else (lt (abs (minus (pc) (match_dup 0))) (const_int 60000)) @@ -7789,7 +7789,7 @@ (define_insn "*icjump_31" [(set_attr "op_type" "RI") (set_attr "type" "branch") (set (attr "length") - (if_then_else (eq (symbol_ref "flag_pic") (const_int 0)) + (if_then_else (not (match_test "flag_pic")) (if_then_else (lt (abs (minus (pc) (match_dup 0))) (const_int 60000)) (const_int 4) (const_int 6)) (if_then_else (lt (abs (minus (pc) (match_dup 0))) (const_int 60000)) @@ -8170,7 +8170,7 @@ (define_insn_and_split "doloop_si31" (set_attr "z10prop" "z10_super_E1") (set_attr "type" "branch") (set (attr "length") - (if_then_else (eq (symbol_ref "flag_pic") (const_int 0)) + (if_then_else (not (match_test "flag_pic")) (if_then_else (lt (abs (minus (pc) (match_dup 0))) (const_int 60000)) (const_int 4) (const_int 6)) (if_then_else (lt (abs (minus (pc) (match_dup 0))) (const_int 60000)) @@ -8282,7 +8282,7 @@ (define_insn "*jump31" [(set_attr "op_type" "RI") (set_attr "type" "branch") (set (attr "length") - (if_then_else (eq (symbol_ref "flag_pic") (const_int 0)) + (if_then_else (not (match_test "flag_pic")) (if_then_else (lt (abs (minus (pc) (match_dup 0))) (const_int 60000)) (const_int 4) (const_int 6)) (if_then_else (lt (abs (minus (pc) (match_dup 0))) (const_int 60000)) @@ -9102,7 +9102,7 @@ (define_insn "main_pool" gcc_unreachable (); } [(set (attr "type") - (if_then_else (ne (symbol_ref "TARGET_CPU_ZARCH") (const_int 0)) + (if_then_else (match_test "TARGET_CPU_ZARCH") (const_string "larl") (const_string "la")))]) (define_insn "reload_base_31"