This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: RFA: Fix lm32 --enable-werror-always build
- From: Richard Henderson <rth at redhat dot com>
- To: Joern Rennecke <amylaar at spamcop dot net>
- Cc: gcc-patches at gcc dot gnu dot org
- Date: Mon, 08 Nov 2010 11:27:30 -0800
- Subject: Re: RFA: Fix lm32 --enable-werror-always build
- References: <20101108103158.s79crfo7gooo0o44-nzlynne@webmail.spamcop.net>
On 11/08/2010 07:31 AM, Joern Rennecke wrote:
> switch (code)
> {
> + rtx temp;
> +
> case LE:
> case LT:
> case LEU:
> case LTU:
> code = swap_condition (code);
> - rtx temp = cmp0;
> + temp = cmp0;
> cmp0 = cmp1;
> cmp1 = temp;
> break;
I'd prefer to see a new block for the case, rather than
injecting into the switch block. I.e.
switch (code)
{
...
case LTU:
{
rtx temp;
...
}
break;
Otherwise ok.
r~