On (Tue) 03 May 2011 [13:03:40], Hans de Goede wrote: > --- > target-i386/kvm.c | 4 ++-- > tcg/tcg.c | 4 ++-- > 2 files changed, 4 insertions(+), 4 deletions(-)
Thanks; just got hit by this. However, there are a couple of whitespace issues: > --- a/tcg/tcg.c > +++ b/tcg/tcg.c > @@ -585,7 +585,7 @@ void tcg_register_helper(void *func, const char *name) > void tcg_gen_callN(TCGContext *s, TCGv_ptr func, unsigned int flags, > int sizemask, TCGArg ret, int nargs, TCGArg *args) > { > -#ifdef TCG_TARGET_I386 > +#if defined TCG_TARGET_I386 && TCG_TARGET_REG_BITS < 64 > int call_type; > #endif > int i; > @@ -612,7 +612,7 @@ void tcg_gen_callN(TCGContext *s, TCGv_ptr func, unsigned > int flags, > > *gen_opc_ptr++ = INDEX_op_call; > nparam = gen_opparam_ptr++; > -#ifdef TCG_TARGET_I386 > +#if defined TCG_TARGET_I386 && TCG_TARGET_REG_BITS < 64 > call_type = (flags & TCG_CALL_TYPE_MASK); > #endif > if (ret != TCG_CALL_DUMMY_ARG) { Both these lines have a trailing space. Care to resubmit with Anthony in CC? You can add: Acked-by: Amit Shah <amit.s...@redhat.com> Amit