Fix ARI violations in c++compile
Commit Message
This patch fixes two violations of the ARI (use of ATTRIBUTE_UNUSED and
"%ll").
gdb/ChangeLog
* compile/compile-cplus-types.c (compile_cplus_debug_output_1): Use
pulongest instead of "%lld".
* compile/compile-cplus-symbols.c (gcc_cplus_convert_symbol): Remove
ATTRIBUTE_UNUSED.
---
gdb/ChangeLog | 7 +++++++
gdb/compile/compile-cplus-symbols.c | 2 +-
gdb/compile/compile-cplus-types.c | 4 ++--
3 files changed, 10 insertions(+), 3 deletions(-)
Comments
>>>>> "Keith" == Keith Seitz <keiths@redhat.com> writes:
Keith> * compile/compile-cplus-types.c (compile_cplus_debug_output_1): Use
Keith> pulongest instead of "%lld".
Keith> * compile/compile-cplus-symbols.c (gcc_cplus_convert_symbol): Remove
Keith> ATTRIBUTE_UNUSED.
Thanks, this is ok.
Tom
On 08/31/2018 07:47 AM, Tom Tromey wrote:
>>>>>> "Keith" == Keith Seitz <keiths@redhat.com> writes:
>
> Keith> * compile/compile-cplus-types.c (compile_cplus_debug_output_1): Use
> Keith> pulongest instead of "%lld".
> Keith> * compile/compile-cplus-symbols.c (gcc_cplus_convert_symbol): Remove
> Keith> ATTRIBUTE_UNUSED.
>
> Thanks, this is ok.
Pushed. Thank you for taking a look.
Keith
@@ -1,3 +1,10 @@
+YYYY-MM-DD Keith Seitz <keiths@redhat.com>
+
+ * compile/compile-cplus-types.c (compile_cplus_debug_output_1): Use
+ pulongest instead of "%lld".
+ * compile/compile-cplus-symbols.c (gcc_cplus_convert_symbol): Remove
+ ATTRIBUTE_UNUSED.
+
2018-08-30 Andrew Burgess <andrew.burgess@embecosm.com>
* riscv-tdep.c (riscv_insn::decode): Decode c.addi4spn, c.sd,
@@ -335,7 +335,7 @@ convert_symbol_bmsym (compile_cplus_instance *instance,
void
gcc_cplus_convert_symbol (void *datum,
struct gcc_cp_context *gcc_context,
- enum gcc_cp_oracle_request request ATTRIBUTE_UNUSED,
+ enum gcc_cp_oracle_request request,
const char *identifier)
{
if (compile_debug)
@@ -1244,9 +1244,9 @@ compile_cplus_instance::gcc_cplus_leave_scope
gcc_decl. */
static void
-compile_cplus_debug_output_1 (gcc_type arg)
+compile_cplus_debug_output_1 (ULONGEST arg)
{
- fprintf_unfiltered (gdb_stdlog, "%lld", arg);
+ fprintf_unfiltered (gdb_stdlog, "%s", pulongest (arg));
}
static void