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 tcg_target_ulong. If you're thinking of the later arithmetic in type tcg_target_ulong, that's fine. Just so long as the sign-extension happened first. r~
