[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] target-alpha: fix subl and s8subl indentation
|
From: |
Stuart Brady |
|
Subject: |
Re: [Qemu-devel] [PATCH] target-alpha: fix subl and s8subl indentation |
|
Date: |
Tue, 18 Mar 2014 21:33:04 +0000 |
|
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Sat, Mar 15, 2014 at 01:40:26PM -0700, Richard Henderson wrote:
> On 03/15/2014 11:33 AM, Paolo Bonzini wrote:
> > Two missing braces, one close and one open, fabulously let the code
> > compile.
> >
> > Signed-off-by: Paolo Bonzini <address@hidden>
>
> Wow. That's ... amazing.
>
> Reviewed-by: Richard Henderson <address@hidden>
The *really* amazing thing about this (at least as far as I can see) is
the missing braces didn't alter the semantics of the code, except that
with the patch we no longer generate an extra TCG op with no effect...
--
Cheers,
Stuart