From 1ba8a9c4ecbfcc849090d1bf08ba244407161789 Mon Sep 17 00:00:00 2001 From: Richard Sandiford Date: Sun, 5 Jan 2003 12:02:24 +0000 Subject: [PATCH] * config/mips/mips.md: Disable the movstrsi define_split. From-SVN: r60911 --- gcc/ChangeLog | 4 ++++ gcc/config/mips/mips.md | 8 +++++++- gcc/testsuite/ChangeLog | 4 ++++ .../gcc.c-torture/execute/20030105-1.c | 20 +++++++++++++++++++ 4 files changed, 35 insertions(+), 1 deletion(-) create mode 100644 gcc/testsuite/gcc.c-torture/execute/20030105-1.c diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 9b984dba57a..cccad464fca 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,7 @@ +2003-01-05 Richard Sandiford + + * config/mips/mips.md: Disable the movstrsi define_split. + 2003-01-05 Kaveh R. Ghazi * alloc-pool.c: Don't include "libiberty.h". diff --git a/gcc/config/mips/mips.md b/gcc/config/mips/mips.md index 3d2d617f69d..3ec54837505 100644 --- a/gcc/config/mips/mips.md +++ b/gcc/config/mips/mips.md @@ -6550,6 +6550,12 @@ move\\t%0,%z4\\n\\ ;; fill a delay slot. This also prevents a bug in delayed branches ;; from showing up, which reuses one of the registers in our clobbers. +;; ??? Disabled because it doesn't preserve alias information for +;; operands 0 and 1. Also, the rtl for the second insn doesn't mention +;; that it uses the registers clobbered by the first. +;; +;; It would probably be better to split the block into individual +;; instructions instead. (define_split [(set (mem:BLK (match_operand:SI 0 "register_operand" "")) (mem:BLK (match_operand:SI 1 "register_operand" ""))) @@ -6561,7 +6567,7 @@ move\\t%0,%z4\\n\\ (use (match_operand:SI 3 "small_int" "")) (use (const_int 0))] - "reload_completed && !TARGET_DEBUG_D_MODE && INTVAL (operands[2]) > 0" + "reload_completed && 0 && INTVAL (operands[2]) > 0" ;; All but the last move [(parallel [(set (mem:BLK (match_dup 0)) diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index ed46afcdc9b..57b4e7310ea 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,7 @@ +2003-01-05 Richard Sandiford + + * gcc.c-torture/execute/20030105-1.c: New test. + 2003-01-04 Kriang Lerdsuwanakij * g++.dg/parse/namespace3.C: Remove extra semicolons. diff --git a/gcc/testsuite/gcc.c-torture/execute/20030105-1.c b/gcc/testsuite/gcc.c-torture/execute/20030105-1.c new file mode 100644 index 00000000000..19f3b2ddb12 --- /dev/null +++ b/gcc/testsuite/gcc.c-torture/execute/20030105-1.c @@ -0,0 +1,20 @@ +int __attribute__ ((noinline)) +foo () +{ + const int a[8] = { 0, 1, 2, 3, 4, 5, 6, 7 }; + int i, sum; + + sum = 0; + for (i = 0; i < sizeof (a) / sizeof (*a); i++) + sum += a[i]; + + return sum; +} + +int +main () +{ + if (foo () != 28) + abort (); + exit (0); +}