[PATCH c++ 0/3] Some C++ patches

[PATCH c++ 0/3] Some C++ patches

[Simon Marchi]

Here are some unrelated C++ patches.  I don't really see a logical order to
send the remaining patches, so my strategy is to try to build and send the
patches that fix the first errors I hit.  Rinse & repeat until done.

Simon Marchi (3):
  arm-linux-tdep: Add explicit cast for int to gdb_syscall conversion
  cris: Use enum bfd_endian to represent endianness
  i386: Fix type of gdbarch_tdep::register_reggroup_p

 gdb/arm-linux-tdep.c | 6 +++---
 gdb/cris-tdep.c      | 2 +-
 gdb/i386-tdep.h      | 2 +-
 3 files changed, 5 insertions(+), 5 deletions(-)

-- 
2.5.1

[PATCH c++ 2/3] cris: Use enum bfd_endian to represent endianness

[Simon Marchi]

gdb/ChangeLog:

	* cris-tdep.c (struct instruction_environment): Change type of
	byte_order to enum bfd_endian.
---
 gdb/cris-tdep.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/gdb/cris-tdep.c b/gdb/cris-tdep.c
index 73e4048..00cf623 100644
--- a/gdb/cris-tdep.c
+++ b/gdb/cris-tdep.c
@@ -485,7 +485,7 @@ struct instruction_environment
   int   delay_slot_pc_active;
   int   xflag_found;
   int   disable_interrupt;
-  int   byte_order;
+  enum bfd_endian byte_order;
 } inst_env_type;
 
 /* Machine-dependencies in CRIS for opcodes.  */
-- 
2.5.1

[PATCH c++ 1/3] arm-linux-tdep: Add explicit cast for int to gdb_syscall conversion

[Simon Marchi]

This patch is taken from Pedro's branch.

gdb/ChangeLog:

	* arm-linux-tdep.c (arm_canonicalize_syscall): Add
	enum gdb_syscall casts.
---
 gdb/arm-linux-tdep.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/gdb/arm-linux-tdep.c b/gdb/arm-linux-tdep.c
index b7c5b99..dc0a8a9 100644
--- a/gdb/arm-linux-tdep.c
+++ b/gdb/arm-linux-tdep.c
@@ -1265,11 +1265,11 @@ arm_canonicalize_syscall (int syscall)
   enum { sys_process_vm_writev = 377 };
 
   if (syscall <= gdb_sys_sched_getaffinity)
-    return syscall;
+    return (enum gdb_syscall) syscall;
   else if (syscall >= 243 && syscall <= 247)
-    return syscall + 2;
+    return (enum gdb_syscall) (syscall + 2);
   else if (syscall >= 248 && syscall <= 253)
-    return syscall + 4;
+    return (enum gdb_syscall) (syscall + 4);
 
   return gdb_sys_no_syscall;
 }
-- 
2.5.1

[PATCH c++ 3/3] i386: Fix type of gdbarch_tdep::register_reggroup_p

[Simon Marchi]

gdb/ChangeLog:

	* i386-tdep.h (struct gdbarch_tdep): Change type of
	register_reggroup_p to gdbarch_register_reggroup_p_ftype.
---
 gdb/i386-tdep.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/gdb/i386-tdep.h b/gdb/i386-tdep.h
index 7880f6c..95288ba 100644
--- a/gdb/i386-tdep.h
+++ b/gdb/i386-tdep.h
@@ -193,7 +193,7 @@ struct gdbarch_tdep
   const struct target_desc *tdesc;
 
   /* Register group function.  */
-  const void *register_reggroup_p;
+  gdbarch_register_reggroup_p_ftype *register_reggroup_p;
 
   /* Offset of saved PC in jmp_buf.  */
   int jb_pc_offset;
-- 
2.5.1

Re: [PATCH c++ 0/3] Some C++ patches

[Pedro Alves]

On 10/09/2015 02:58 PM, Simon Marchi wrote:
> Here are some unrelated C++ patches.  I don't really see a logical order to
> send the remaining patches, so my strategy is to try to build and send the
> patches that fix the first errors I hit.  Rinse & repeat until done.

LGTM.

Thanks,
Pedro Alves

Re: [PATCH c++ 0/3] Some C++ patches

[Simon Marchi]

On 15-10-09 10:09 AM, Pedro Alves wrote:
> LGTM.

All pushed, thanks!