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 jan.kratochvil@redhat.com
Sun Jan 18 07:01:00 GMT 2015


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



More information about the Gdb-patches mailing list