[binutils-gdb] [PowerPC] Remove rs6000_pseudo_register_reggroup_p

[Pedro Franco de Carvalho <pedromfc at sourceware dot org>]

https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=500f01a0e2f79d64cc08011ad70c6ffa1e1c7694

commit 500f01a0e2f79d64cc08011ad70c6ffa1e1c7694
Author: Pedro Franco de Carvalho <pedromfc@linux.ibm.com>
Date:   Fri Oct 26 09:37:53 2018 -0300

    [PowerPC] Remove rs6000_pseudo_register_reggroup_p
    
    This patch removes rs6000_pseudo_register_reggroup_p.
    
    Group membership for the pseudoregisters can be detected through their
    types in default_register_reggroup_p through
    tdesc_register_reggroup_p.
    
    gdb/ChangeLog:
    2018-10-26  Pedro Franco de Carvalho  <pedromfc@linux.ibm.com>
    
    	* rs6000-tdep.c: Remove reggroups.h include.
    	(rs6000_pseudo_register_reggroup_p): Remove.
    	(rs6000_gdbarch_init): Remove call to
    	set_tdesc_pseudo_register_reggroup_p.

Diff:
---
 gdb/ChangeLog     |  7 +++++++
 gdb/rs6000-tdep.c | 24 ------------------------
 2 files changed, 7 insertions(+), 24 deletions(-)

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index e170dd0..ba06946 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,5 +1,12 @@
 2018-10-26  Pedro Franco de Carvalho  <pedromfc@linux.ibm.com>
 
+	* rs6000-tdep.c: Remove reggroups.h include.
+	(rs6000_pseudo_register_reggroup_p): Remove.
+	(rs6000_gdbarch_init): Remove call to
+	set_tdesc_pseudo_register_reggroup_p.
+
+2018-10-26  Pedro Franco de Carvalho  <pedromfc@linux.ibm.com>
+
 	* reggroups.c (default_register_reggroup_p): Return true for
 	decfloat registers and float_reggroup.
 
diff --git a/gdb/rs6000-tdep.c b/gdb/rs6000-tdep.c
index 80d8504..c967bc7 100644
--- a/gdb/rs6000-tdep.c
+++ b/gdb/rs6000-tdep.c
@@ -36,7 +36,6 @@
 #include "infcall.h"
 #include "sim-regno.h"
 #include "gdb/sim-ppc.h"
-#include "reggroups.h"
 #include "dwarf2-frame.h"
 #include "target-descriptions.h"
 #include "user-regs.h"
@@ -2408,27 +2407,6 @@ rs6000_pseudo_register_type (struct gdbarch *gdbarch, int regnum)
     return builtin_type (gdbarch)->builtin_double;
 }
 
-/* Is REGNUM a member of REGGROUP?  */
-static int
-rs6000_pseudo_register_reggroup_p (struct gdbarch *gdbarch, int regnum,
-				   struct reggroup *group)
-{
-  struct gdbarch_tdep *tdep = gdbarch_tdep (gdbarch);
-
-  /* These are the only pseudo-registers we support.  */
-  gdb_assert (IS_SPE_PSEUDOREG (tdep, regnum)
-	      || IS_DFP_PSEUDOREG (tdep, regnum)
-	      || IS_VSX_PSEUDOREG (tdep, regnum)
-	      || IS_EFP_PSEUDOREG (tdep, regnum));
-
-  /* These are the e500 pseudo-registers or the POWER7 VSX registers.  */
-  if (IS_SPE_PSEUDOREG (tdep, regnum) || IS_VSX_PSEUDOREG (tdep, regnum))
-    return group == all_reggroup || group == vector_reggroup;
-  else
-    /* PPC decimal128 or Extended FP pseudo-registers.  */
-    return group == all_reggroup || group == float_reggroup;
-}
-
 /* The register format for RS/6000 floating point registers is always
    double, we need a conversion if the memory format is float.  */
 
@@ -6439,8 +6417,6 @@ rs6000_gdbarch_init (struct gdbarch_info info, struct gdbarch_list *arches)
     }
 
   set_tdesc_pseudo_register_type (gdbarch, rs6000_pseudo_register_type);
-  set_tdesc_pseudo_register_reggroup_p (gdbarch,
-					rs6000_pseudo_register_reggroup_p);
   tdesc_use_registers (gdbarch, tdesc, tdesc_data);
 
   /* Override the normal target description method to make the SPE upper