Patchwork [H8300] Use match_test rather than eq/ne symbol_ref

login
register
mail settings
Submitter Richard Sandiford
Date Sept. 13, 2011, 5:50 p.m.
Message ID <874o0g8h7c.fsf@firetop.home>
Download mbox | patch
Permalink /patch/114517/
State New
Headers show

Comments

Richard Sandiford - Sept. 13, 2011, 5:50 p.m.
As per the subject.  Tested by making sure that there were no new
warnings building h8300-elf, and that there were no changes in the
assembly output for the C and C++ testsuite.  OK to install?

Richard


gcc/
	* config/h8300/h8300.md: Use match_test rather than eq/ne symbol_ref
	throughout file.
Jeff Law - Sept. 13, 2011, 5:55 p.m.
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 09/13/11 11:50, Richard Sandiford wrote:
> As per the subject.  Tested by making sure that there were no new 
> warnings building h8300-elf, and that there were no changes in the 
> assembly output for the C and C++ testsuite.  OK to install?
> 
> Richard
> 
> 
> gcc/ * config/h8300/h8300.md: Use match_test rather than eq/ne
> symbol_ref throughout file.
OK
jeff
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.11 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/

iQEcBAEBAgAGBQJOb5kMAAoJEBRtltQi2kC7JhUIAImGzTwx0yllpWVE53L6UZPI
UT3IdwgFjBdSfpU5RjXAmOqFUj6xZKHVtajM0e8h28TG43X+iBaqB59NGFUk30d4
38To54t/3cVZYTkg6brunXbre4rGODdvB8UNROIj4WSHeugjvNme4l8W9W7nd/6m
pIFrz0msvMOpkj49o9t8DdkVWyQocZr7VOuLtIdevWio3ZWQzs6dVjPV9NmBHBtj
Eo4FWi0c8yH5Heoem5INeVncSUH3Pw+EP5JheisEUaRXPU+NtxhtOKJtZU0kQj+9
ODkTVO5JqdwKriKEMAlXwR7nmsQ1YePZgX5wntHzDCI3YW8hEgRo/jysbAwwtwU=
=j7YL
-----END PGP SIGNATURE-----

Patch

Index: gcc/config/h8300/h8300.md
===================================================================
--- gcc/config/h8300/h8300.md	2011-09-13 18:43:40.000000000 +0100
+++ gcc/config/h8300/h8300.md	2011-09-13 18:49:19.000000000 +0100
@@ -171,9 +171,9 @@  (define_delay (and (eq_attr "delay_slot"
 
 (define_asm_attributes
   [(set (attr "length")
-	(cond [(ne (symbol_ref "TARGET_H8300")  (const_int 0)) (const_int 4)
-	       (ne (symbol_ref "TARGET_H8300H") (const_int 0)) (const_int 10)
-	       (ne (symbol_ref "TARGET_H8300S") (const_int 0)) (const_int 10)]
+	(cond [(match_test "TARGET_H8300") (const_int 4)
+	       (match_test "TARGET_H8300H") (const_int 10)
+	       (match_test "TARGET_H8300S") (const_int 10)]
 	      (const_int 14)))])
 
 (include "predicates.md")
@@ -2416,7 +2416,7 @@  (define_insn "jump"
 }"
   [(set_attr "type" "branch")
    (set (attr "delay_slot")
-	(if_then_else (ne (symbol_ref "TARGET_H8300SX") (const_int 0))
+	(if_then_else (match_test "TARGET_H8300SX")
 		      (const_string "jump")
 		      (const_string "none")))
    (set_attr "cc" "none")])