This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: Constification patch (0/5)
- To: gcc-patches at gcc dot gnu dot org
- Subject: Re: Constification patch (0/5)
- From: Zack Weinberg <zack at wolery dot cumb dot org>
- Date: Tue, 9 May 2000 14:51:27 -0700
- References: <20000509124423.A21854@wolery.cumb.org>
On Tue, May 09, 2000 at 12:44:23PM -0700, Zack Weinberg wrote:
> The upcoming patch, in five pieces, turns on -Wwrite-strings and
> corrects all but one of the resulting warnings.
This broke bootstrap on targets that use multilibs. Geoff Keating's
autobuilder caught that - thanks!
I can't bootstrap --target=powerpc-eabisim completely, right now,
because of the ICE in add_insn_after - but this patch should correct
the problem that's my fault.
zw
* gcc.c (used_arg): Skip over a semicolon at the end of the
split-up loop; don't break out of it.
===================================================================
Index: gcc.c
--- gcc.c 2000/05/09 19:55:47 1.142
+++ gcc.c 2000/05/09 21:49:50
@@ -5993,8 +5993,8 @@ used_arg (p, len)
}
matches[i].rep_len = q - matches[i].replace;
i++;
- if (*q != ';')
- break;
+ if (*q == ';')
+ q++;
}
/* Now build a list of the replacement string for switches that we care