Hi
We had two warnings on 32bit targets. With this change I am able to
compile with -Werror on:
- linux 64bit
- linux 32bit
- windows 32bit (cross-compile with Fedora mingw64)
- windows 64bit (cross-compile with Fedora mingw64)
I am lying in this one, really my crosscompiler for windows 64 bit barf at
CXX disas/libvixl/a64/disasm-a64.o
/mnt/kvm/qemu/next/disas/libvixl/a64/disasm-a64.cc: In member function 'virtual
void vixl::Disassembler::AppendPCRelativeOffsetToOutput(const
vixl::Instruction*, int64_t)':
/mnt/kvm/qemu/next/disas/libvixl/a64/disasm-a64.cc:1365:57: error: unknown
conversion type character 'l' in format [-Werror=format=]
AppendToOutput("#%c0x%" PRIx64, sign, std::abs(offset));
^
/mnt/kvm/qemu/next/disas/libvixl/a64/disasm-a64.cc:1365:57: error: too many
arguments for format [-Werror=format-extra-args]
/mnt/kvm/qemu/next/disas/libvixl/a64/disasm-a64.cc: In member function 'virtual
void vixl::Disassembler::AppendAddressToOutput(const vixl::Instruction*, const
void*)':
/mnt/kvm/qemu/next/disas/libvixl/a64/disasm-a64.cc:1372:76: error: unknown
conversion type character 'l' in format [-Werror=format=]
AppendToOutput("(addr 0x%" PRIxPTR ")", reinterpret_cast<uintptr_t>(addr));
^
/mnt/kvm/qemu/next/disas/libvixl/a64/disasm-a64.cc:1372:76: error: too many
arguments for format [-Werror=format-extra-args]
/mnt/kvm/qemu/next/disas/libvixl/a64/disasm-a64.cc: In member function 'virtual
void vixl::Disassembler::AppendCodeRelativeAddressToOutput(const
vixl::Instruction*, const void*)':
/mnt/kvm/qemu/next/disas/libvixl/a64/disasm-a64.cc:1393:52: error: unknown
conversion type character 'l' in format [-Werror=format=]
AppendToOutput("(addr 0x%" PRIx64 ")", rel_addr);
^
/mnt/kvm/qemu/next/disas/libvixl/a64/disasm-a64.cc:1393:52: error: too many
arguments for format [-Werror=format-extra-args]
/mnt/kvm/qemu/next/disas/libvixl/a64/disasm-a64.cc:1395:54: error: unknown
conversion type character 'l' in format [-Werror=format=]
AppendToOutput("(addr -0x%" PRIx64 ")", -rel_addr);
^
.....
I am wondering if it really understand PRI*64 at all.
Please, review.
Juan.
Juan Quintela (2):
migration: print ram_addr_t as RAM_ADDR_FMT not %zx
migration: Make 32bit linux compile with RDMA
migration/ram.c | 5 +++--
migration/rdma.c | 2 +-
2 files changed, 4 insertions(+), 3 deletions(-)
--
2.5.0
