[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH for-2.5 03/10] tcg/optimize: track const/copy st
From: |
Aurelien Jarno |
Subject: |
Re: [Qemu-devel] [PATCH for-2.5 03/10] tcg/optimize: track const/copy status separately |
Date: |
Wed, 29 Jul 2015 18:25:21 +0200 |
User-agent: |
Mutt/1.5.23 (2014-03-12) |
On 2015-07-29 17:10, Alex Bennée wrote:
>
> Aurelien Jarno <address@hidden> writes:
>
> > Use two bools to track constants and copies instead of an enum.
>
> More of an explanation would be useful as to why, otherwise it seems to
> me we are just increasing the size of the structure (assuming the
> compiler uses the same base sizes for the bool and enum).
The reason is in the following patch, to allow separate tracking of const
and copy. Note that this doesn't increase the structure size due to
alignment and that the v2 of this patchset actually uses only one bool.
I'll add a note about why it is need for v3.
--
Aurelien Jarno GPG: 4096R/1DDD8C9B
address@hidden http://www.aurel32.net
- [Qemu-devel] [PATCH for-2.5 08/10] tcg/optimize: add optimizations for ext_i32_i64 and extu_i32_i64 ops, (continued)
- [Qemu-devel] [PATCH for-2.5 08/10] tcg/optimize: add optimizations for ext_i32_i64 and extu_i32_i64 ops, Aurelien Jarno, 2015/07/24
- [Qemu-devel] [PATCH for-2.5 05/10] tcg: rename trunc_shr_i32 into trunc_shr_i64_i32, Aurelien Jarno, 2015/07/24
- [Qemu-devel] [PATCH for-2.5 02/10] tcg/optimize: add temp_is_const and temp_is_copy functions, Aurelien Jarno, 2015/07/24
- [Qemu-devel] [PATCH for-2.5 03/10] tcg/optimize: track const/copy status separately, Aurelien Jarno, 2015/07/24
- [Qemu-devel] [PATCH for-2.5 01/10] tcg/optimize: optimize temps tracking, Aurelien Jarno, 2015/07/24
- [Qemu-devel] [PATCH for-2.5 07/10] tcg: implement real ext_i32_i64 and extu_i32_i64 ops, Aurelien Jarno, 2015/07/24