[Qemu-devel] [PATCH 1/5] tci: Use 32-bit signed offsets to loads/stores

2013-04-11 Thread Stefan Weil
From: Richard Henderson Since the change to tcg_exit_req, the first insn of every TB is a load with a negative offset from env. Signed-off-by: Richard Henderson Signed-off by: Stefan Weil --- tcg/tci/tcg-target.c |4 ++-- tci.c| 36 ++-- 2

[Qemu-devel] [PATCH 1/5] tci: Use 32-bit signed offsets to loads/stores

2013-03-28 Thread Richard Henderson
Since the change to tcg_exit_req, the first insn of every TB is a load with a negative offset from env. Signed-off-by: Richard Henderson --- tcg/tci/tcg-target.c | 4 ++-- tci.c| 36 ++-- 2 files changed, 24 insertions(+), 16 deletions(-) diff --

Re: [Qemu-devel] [PATCH 1/5] tci: Use 32-bit signed offsets to loads/stores

2013-03-28 Thread Richard Henderson
On 03/28/2013 08:45 AM, Stefan Weil wrote: >> > -t2 = tci_read_i32(&tb_ptr); >> > +t2 = tci_read_s32(&tb_ptr); > I'm afraid that old and new generated code are identical, > because t2 is an unsigned tcg_target_ulong. No it's not, because s32 is sign-extended from int32_t to

Re: [Qemu-devel] [PATCH 1/5] tci: Use 32-bit signed offsets to loads/stores

2013-03-28 Thread Stefan Weil
Am 28.03.2013 16:37, schrieb Richard Henderson: > Since the change to tcg_exit_req, the first insn of every TB is > a load with a negative offset from env. > > Signed-off-by: Richard Henderson > --- > tcg/tci/tcg-target.c | 4 ++-- > tci.c| 36 ++--