As per the subject.  Tested by making sure that there were no new
warnings building v850-elf, and that there were no changes in the
assembly output for the C and C++ testsuite.  OK to install?

Richard


gcc/
        * config/v850/v850.md: Use match_test rather than eq/ne symbol_ref
        throughout file.

Index: gcc/config/v850/v850.md
===================================================================
--- gcc/config/v850/v850.md     2011-09-13 18:43:39.000000000 +0100
+++ gcc/config/v850/v850.md     2011-09-13 19:06:27.000000000 +0100
@@ -61,15 +61,15 @@ (define_attr "type" "load,store,bit1,mul
   (const_string "other"))
 
 (define_attr "cpu" "none,v850,v850e,v850e1,v850e2,v850e2v3"
-  (cond [(ne (symbol_ref "TARGET_V850") (const_int 0))
+  (cond [(match_test "TARGET_V850")
        (const_string "v850")
-       (ne (symbol_ref "TARGET_V850E") (const_int 0))
+       (match_test "TARGET_V850E")
        (const_string "v850e")
-       (ne (symbol_ref "TARGET_V850E1") (const_int 0))
+       (match_test "TARGET_V850E1")
        (const_string "v850e1")
-       (ne (symbol_ref "TARGET_V850E2") (const_int 0))
+       (match_test "TARGET_V850E2")
        (const_string "v850e2")
-       (ne (symbol_ref "TARGET_V850E2") (const_int 0))
+       (match_test "TARGET_V850E2")
        (const_string "v850e2v3")]
        (const_string "none")))
 
@@ -2460,7 +2460,7 @@ (define_insn "save_interrupt"
     }
 }"
   [(set (attr "length")
-        (if_then_else (ne (symbol_ref "TARGET_LONG_CALLS") (const_int 0))
+        (if_then_else (match_test "TARGET_LONG_CALLS")
                        (const_int 10)
                        (const_int 34)))
    (set_attr "cc" "clobber")])
@@ -2492,7 +2492,7 @@ (define_insn "return_interrupt"
     }
 }"
   [(set (attr "length")
-        (if_then_else (ne (symbol_ref "TARGET_LONG_CALLS") (const_int 0))
+        (if_then_else (match_test "TARGET_LONG_CALLS")
                        (const_int 4)
                        (const_int 24)))
    (set_attr "cc" "clobber")])
@@ -2586,7 +2586,7 @@ (define_insn "save_all_interrupt"
   return \"\";
 }"
   [(set (attr "length")
-        (if_then_else (ne (symbol_ref "TARGET_LONG_CALLS") (const_int 0))
+        (if_then_else (match_test "TARGET_LONG_CALLS")
                        (const_int 4)
                        (const_int 62)
        ))
@@ -2685,7 +2685,7 @@ (define_insn "restore_all_interrupt"
   return \"\";
 }"
   [(set (attr "length")
-        (if_then_else (ne (symbol_ref "TARGET_LONG_CALLS") (const_int 0))
+        (if_then_else (match_test "TARGET_LONG_CALLS")
                        (const_int 4)
                        (const_int 62)
        ))

Reply via email to