[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v1 1/2] tcg: Add support for constant value prom
From: |
Richard Henderson |
Subject: |
Re: [Qemu-devel] [PATCH v1 1/2] tcg: Add support for constant value promises |
Date: |
Fri, 15 Jan 2016 10:20:04 -0800 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.5.0 |
On 01/15/2016 07:35 AM, Lluís Vilanova wrote:
> +TCGv_i64 tcg_promise_i64(TCGv_promise_i64 *promise)
> +{
> + int pi = tcg_ctx.gen_next_parm_idx;
> + *promise = (TCGv_promise_i64)&tcg_ctx.gen_opparam_buf[pi];
> + return tcg_const_i64(0xdeadcafe);
> +}
This doesn't work for a 32-bit host. The constant may be split across two
different parameter indices, and you don't know exactly where the second will
be.
Because of that, I think this is over-engineered, and really prefer the simpler
interface that Edgar posted last week.
r~
- [Qemu-devel] [PATCH v1 0/2] tcg: Add support for constant value promises, Lluís Vilanova, 2016/01/15
- [Qemu-devel] [PATCH v1 1/2] tcg: Add support for constant value promises, Lluís Vilanova, 2016/01/15
- Re: [Qemu-devel] [PATCH v1 1/2] tcg: Add support for constant value promises,
Richard Henderson <=
- Re: [Qemu-devel] [PATCH v1 1/2] tcg: Add support for constant value promises, Lluís Vilanova, 2016/01/15
- Re: [Qemu-devel] [PATCH v1 1/2] tcg: Add support for constant value promises, Richard Henderson, 2016/01/15
- Re: [Qemu-devel] [PATCH v1 1/2] tcg: Add support for constant value promises, Lluís Vilanova, 2016/01/16
- Re: [Qemu-devel] [PATCH v1 1/2] tcg: Add support for constant value promises, Edgar E. Iglesias, 2016/01/19
- Re: [Qemu-devel] [PATCH v1 1/2] tcg: Add support for constant value promises, Lluís Vilanova, 2016/01/19
[Qemu-devel] [PATCH v1 2/2] gen-icount: Use constant value promises, Lluís Vilanova, 2016/01/15