This is the mail archive of the gdb-cvs@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

[binutils-gdb] Construct readonly regcache without address space


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

commit f26ae15b471aaddee81d9d6c03af1cb0f2081735
Author: Yao Qi <yao.qi@linaro.org>
Date:   Thu Nov 2 15:15:42 2017 +0000

    Construct readonly regcache without address space
    
    The address space is useless to readonly regcache, so this patch removes
    the parameter to construct readonly regcache.
    
    address_space was added in regcache by 6c95b8d, but for read-write
    regcache.  regcache::aspace is used for various breakpoint/watchpoint
    checking, and these regcache are not read-only regcache.
    
    gdb:
    
    2017-11-02  Yao Qi  <yao.qi@linaro.org>
    
    	* frame.c (do_frame_register_read): Remove aspace.
    	* jit.c (jit_frame_sniffer): Likwise.
    	* ppc-linux-tdep.c (ppu2spu_sniffer): Likewise.
    	* regcache.c (regcache::regcache): Pass nullptr.
    	(regcache_print): Caller updated.
    	* regcache.h (regcache::regcache): Remove one constructor
    	parameter aspace.

Diff:
---
 gdb/ChangeLog        | 10 ++++++++++
 gdb/frame.c          |  3 +--
 gdb/jit.c            |  4 +---
 gdb/ppc-linux-tdep.c |  5 ++---
 gdb/regcache.c       |  4 ++--
 gdb/regcache.h       |  4 ++--
 6 files changed, 18 insertions(+), 12 deletions(-)

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 7cec806..085b615 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,5 +1,15 @@
 2017-11-02  Yao Qi  <yao.qi@linaro.org>
 
+	* frame.c (do_frame_register_read): Remove aspace.
+	* jit.c (jit_frame_sniffer): Likwise.
+	* ppc-linux-tdep.c (ppu2spu_sniffer): Likewise.
+	* regcache.c (regcache::regcache): Pass nullptr.
+	(regcache_print): Caller updated.
+	* regcache.h (regcache::regcache): Remove one constructor
+	parameter aspace.
+
+2017-11-02  Yao Qi  <yao.qi@linaro.org>
+
 	* regcache.h (regcache) <m_readonly_p>: Change it to const bool.
 
 2017-11-02  Yao Qi  <yao.qi@linaro.org>
diff --git a/gdb/frame.c b/gdb/frame.c
index e40685f..e643823 100644
--- a/gdb/frame.c
+++ b/gdb/frame.c
@@ -1020,9 +1020,8 @@ do_frame_register_read (void *src, int regnum, gdb_byte *buf)
 std::unique_ptr<struct regcache>
 frame_save_as_regcache (struct frame_info *this_frame)
 {
-  const address_space *aspace = get_frame_address_space (this_frame);
   std::unique_ptr<struct regcache> regcache
-    (new struct regcache (get_frame_arch (this_frame), aspace));
+    (new struct regcache (get_frame_arch (this_frame)));
 
   regcache_save (regcache.get (), do_frame_register_read, this_frame);
   return regcache;
diff --git a/gdb/jit.c b/gdb/jit.c
index b70f87c..7538684 100644
--- a/gdb/jit.c
+++ b/gdb/jit.c
@@ -1199,13 +1199,11 @@ jit_frame_sniffer (const struct frame_unwind *self,
 
   gdb_assert (!*cache);
 
-  const address_space *aspace = get_frame_address_space (this_frame);
-
   gdbarch = get_frame_arch (this_frame);
 
   *cache = XCNEW (struct jit_unwind_private);
   priv_data = (struct jit_unwind_private *) *cache;
-  priv_data->regcache = new regcache (gdbarch, aspace);
+  priv_data->regcache = new regcache (gdbarch);
   priv_data->this_frame = this_frame;
 
   callbacks.priv_data = priv_data;
diff --git a/gdb/ppc-linux-tdep.c b/gdb/ppc-linux-tdep.c
index d01cff8..847908a 100644
--- a/gdb/ppc-linux-tdep.c
+++ b/gdb/ppc-linux-tdep.c
@@ -1361,10 +1361,9 @@ ppu2spu_sniffer (const struct frame_unwind *self,
 	{
 	  struct ppu2spu_cache *cache
 	    = FRAME_OBSTACK_CALLOC (1, struct ppu2spu_cache);
-
-	  const address_space *aspace = get_frame_address_space (this_frame);
 	  std::unique_ptr<struct regcache> regcache
-	    (new struct regcache (data.gdbarch, aspace));
+	    (new struct regcache (data.gdbarch));
+
 	  regcache_save (regcache.get (), ppu2spu_unwind_register, &data);
 
 	  cache->frame_id = frame_id_build (base, func);
diff --git a/gdb/regcache.c b/gdb/regcache.c
index ef05168..571aba1 100644
--- a/gdb/regcache.c
+++ b/gdb/regcache.c
@@ -211,7 +211,7 @@ do_cooked_read (void *src, int regnum, gdb_byte *buf)
 }
 
 regcache::regcache (readonly_t, const regcache &src)
-  : regcache (src.arch (), src.aspace (), true)
+  : regcache (src.arch (), nullptr, true)
 {
   gdb_assert (!src.m_readonly_p);
   save (do_cooked_read, (void *) &src);
@@ -1574,7 +1574,7 @@ regcache_print (const char *args, enum regcache_dump_what what_to_dump)
       /* For the benefit of "maint print registers" & co when
 	 debugging an executable, allow dumping a regcache even when
 	 there is no thread selected / no registers.  */
-      regcache dummy_regs (target_gdbarch (), nullptr);
+      regcache dummy_regs (target_gdbarch ());
       dummy_regs.dump (out, what_to_dump);
     }
 }
diff --git a/gdb/regcache.h b/gdb/regcache.h
index 05d96b3..c5ef41b 100644
--- a/gdb/regcache.h
+++ b/gdb/regcache.h
@@ -232,8 +232,8 @@ typedef struct cached_reg
 class regcache
 {
 public:
-  regcache (gdbarch *gdbarch, const address_space *aspace_)
-    : regcache (gdbarch, aspace_, true)
+  regcache (gdbarch *gdbarch)
+    : regcache (gdbarch, nullptr, true)
   {}
 
   struct readonly_t {};


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]