This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Blackfin patch: Correct addrtype attributes for push and pop
- From: Bernd Schmidt <bernds_cb1 at t-online dot de>
- To: GCC Patches <gcc-patches at gcc dot gnu dot org>
- Date: Fri, 01 Dec 2006 18:55:00 +0100
- Subject: Blackfin patch: Correct addrtype attributes for push and pop
This patch corrects errors in the pushsi and popsi patterns; we weren't
setting the correct addrtype attribute in all cases.
Committed as 119408.
Bernd
Index: ChangeLog
===================================================================
--- ChangeLog (revision 119407)
+++ ChangeLog (working copy)
@@ -1,3 +1,8 @@
+2006-12-01 Bernd Schmidt <bernd.schmidt@analog.com>
+
+ * config/bfin/bfin.md (pushsi_insn, popsi_insn): Set correct addrtype
+ attribute.
+
2006-12-01 Richard Guenther <rguenther@suse.de>
PR c++/29433
Index: config/bfin/bfin.md
===================================================================
--- config/bfin/bfin.md (revision 119407)
+++ config/bfin/bfin.md (working copy)
@@ -502,15 +502,16 @@ (define_insn "*pushsi_insn"
""
"[--SP] = %0;"
[(set_attr "type" "mcst")
+ (set_attr "addrtype" "32bit")
(set_attr "length" "2")])
(define_insn "*popsi_insn"
- [(set (match_operand:SI 0 "register_operand" "=xy")
+ [(set (match_operand:SI 0 "register_operand" "=d,xy")
(mem:SI (post_inc:SI (reg:SI REG_SP))))]
""
"%0 = [SP++];"
[(set_attr "type" "mcld")
- (set_attr "addrtype" "preg")
+ (set_attr "addrtype" "preg,32bit")
(set_attr "length" "2")])
;; The first alternative is used to make reload choose a limited register