aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorTom de Vries <tdevries@suse.de>2025-04-30 09:49:58 +0200
committerTom de Vries <tdevries@suse.de>2025-04-30 09:49:58 +0200
commit5f02f0fae18c996541fb40c85bd98d1a5ab990dd (patch)
treec89492d1bc802dd5034e7bd17bdd1c0c3ce2b055
parent10ec3d5de0ee75ad56029bfa6b5d3be99f4eac3c (diff)
downloadbinutils-5f02f0fae18c996541fb40c85bd98d1a5ab990dd.zip
binutils-5f02f0fae18c996541fb40c85bd98d1a5ab990dd.tar.gz
binutils-5f02f0fae18c996541fb40c85bd98d1a5ab990dd.tar.bz2
[gdb/build] Fix Wformat errors in gdb/riscv-tdep.c
When building gdb with --enable-targets=all on arm-linux, I run into: ... gdb/riscv-tdep.c: In function ‘bool try_read(regcache*, int, ULONGEST&)’: gdb/riscv-tdep.c:4887:18: error: format ‘%lx’ expects argument of type \ ‘long unsigned int’, but argument 2 has type ‘ULONGEST’ \ {aka ‘long long unsigned int’} [-Werror=format=] 4887 | warning (_("Can not read at address %lx"), addr); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ... and a few more Wformat errors, due to commit b9c7eed0c24 ("This commit adds record full support for rv64gc instruction set"). Fix these by using hex_string. Tested by completing a build on arm-linux.
-rw-r--r--gdb/riscv-tdep.c13
1 files changed, 7 insertions, 6 deletions
diff --git a/gdb/riscv-tdep.c b/gdb/riscv-tdep.c
index a735c09..8998a29 100644
--- a/gdb/riscv-tdep.c
+++ b/gdb/riscv-tdep.c
@@ -4884,7 +4884,7 @@ try_read (struct regcache *regcache, int regnum, ULONGEST &addr)
if (regcache->raw_read (regnum, &addr)
!= register_status::REG_VALID)
{
- warning (_("Can not read at address %lx"), addr);
+ warning (_("Can not read at address %s"), hex_string (addr));
return false;
}
return true;
@@ -5270,8 +5270,8 @@ private:
|| try_save_pc_rd_mem (ival, regcache))
return !has_error ();
- warning (_("Currently this instruction with len 4(%lx) is unsupported"),
- ival);
+ warning (_("Currently this instruction with len 4(%s) is unsupported"),
+ hex_string (ival));
return false;
}
@@ -5380,8 +5380,8 @@ private:
|| !save_mem (addr + offset, 4) || set_ordinary_record_type ());
}
- warning (_("Currently this instruction with len 2(%lx) is unsupported"),
- ival);
+ warning (_("Currently this instruction with len 2(%s) is unsupported"),
+ hex_string (ival));
return false;
}
@@ -5415,7 +5415,8 @@ public:
are not defined yet, so just ignore it. */
gdb_assert (m_length > 0 && m_length % 2 == 0);
- warning (_("Can not record unknown instruction (opcode = %lx)"), ival);
+ warning (_("Can not record unknown instruction (opcode = %s)"),
+ hex_string (ival));
return false;
}