Re: Broken build: rs6000-tdep.c: 32-bit host --enable-targets=all --enable-64-bit-bfd [Re: [PATCH 2/3 v4] Process record support for PowerPC]

Jan Kratochvil
2015-01-18T07:01:36Z

On Sun, 18 Jan 2015 05:46:58 +0100, Wei-cheng Wang wrote:
> Here is the fix. Ok to commit?
[...]
> + fprintf_unfiltered (gdb_stdlog, "Warning: Don't know how to record %08x "
> + "at 0x%lx, 4-%d.\n", insn, (unsigned long) addr, ext);

That AFAIK does not work on 64-bit MS-Windows where CORE_ADDR is 64-bit but
long is 32-bit there.

GDB has paddress(), core_addr_to_string() and hex_string() instead while the
codebase uses these three interchangeably. But the codebase uses even the
%lx+long form.

Jan