On 03/24/2014 07:05 AM, Claudio Fontana wrote:
>> > + /* Look for host pointer values within 4G of the PC. This happens
>> > + often when loading pointers to QEMU's own data structures. */
>> > + disp = (value >> 12) - ((intptr_t)s->code_ptr >> 12);
>> > + if (disp == sextract64(disp, 0, 21)) {
>
> nit.. for the check to be correct in all cases, the assumption here is that
> intptr_t is the same size as a signed target long; would a cast to
> tcg_target_long instead of intptr_t be "safer"?
>
I don't think so.
Gcc 4.9 supports an -m32 abi for aarch64. Suppose we were to compile qemu this
way. In that case tcg_target_long would be larger than intptr_t, and the cast
here would Werror. But leaving it intptr_t, we get a proper sign-extension
with type promotion to tcg_target_long, and the arithmetic will in fact work
like expected.
r~
