Re: [Qemu-devel] [PATCH 1/2] tcg: Fix regression in tcg_gen_deposit_i64.

[malc]

On Tue, 1 Nov 2011, Richard Henderson wrote:

> The error being caused by the failure to copy the other half of
> the input to the output after having narrowed the deposit operation.

Thanks, applied.

> 
> Signed-off-by: Richard Henderson <address@hidden>
> ---
>  tcg/tcg-op.h |    2 ++
>  1 files changed, 2 insertions(+), 0 deletions(-)
> 
> diff --git a/tcg/tcg-op.h b/tcg/tcg-op.h
> index 24ec7fc..8637fe8 100644
> --- a/tcg/tcg-op.h
> +++ b/tcg/tcg-op.h
> @@ -2090,6 +2090,7 @@ static inline void tcg_gen_deposit_i64(TCGv_i64 ret, 
> TCGv_i64 arg1,
>  
>  #if TCG_TARGET_REG_BITS == 32
>      if (ofs >= 32) {
> +        tcg_gen_mov_i32(TCGV_LOW(ret), TCGV_LOW(arg1));
>          tcg_gen_deposit_i32(TCGV_HIGH(ret), TCGV_HIGH(arg1),
>                              TCGV_LOW(arg2), ofs - 32, len);
>          return;
> @@ -2097,6 +2098,7 @@ static inline void tcg_gen_deposit_i64(TCGv_i64 ret, 
> TCGv_i64 arg1,
>      if (ofs + len <= 32) {
>          tcg_gen_deposit_i32(TCGV_LOW(ret), TCGV_LOW(arg1),
>                              TCGV_LOW(arg2), ofs, len);
> +        tcg_gen_mov_i32(TCGV_HIGH(ret), TCGV_HIGH(arg1));
>          return;
>      }
>  #endif
> 

-- 
mailto:address@hidden