qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH v3 7/9] tcg: Compress dead_temps and mem_temps i


From: Richard Henderson
Subject: Re: [Qemu-devel] [PATCH v3 7/9] tcg: Compress dead_temps and mem_temps into a single array
Date: Wed, 3 Aug 2016 23:52:31 +0530
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.1.1

On 07/25/2016 08:45 PM, Aurelien Jarno wrote:
> +#define TS_DEAD  1
> +#define TS_SYNC  2
I am not sure that TS_SYNC is the best name for that. There we really
want to tell that at this moment in the TCG instruction stream the
operand is in memory. It doesn't implied it is synced. Maybe TS_MEM?

The rest looks fine to me. The other alternative would have been to use
the TCGTempSet with the bitmap functions like in optimize.c, and use
only 2 bits per temp. That something that can be done later though.

I changed the name to TS_MEM as suggested.


r~




reply via email to

[Prev in Thread] Current Thread [Next in Thread]