|
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 2I 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~
[Prev in Thread] | Current Thread | [Next in Thread] |