gdb: refactor test_get_thread_arch_aspace_regcache

Do these misc changes to test_get_thread_arch_aspace_regcache:

- Rename to get_thread_arch_aspace_regcache_and_check.  The following
  patch introduces a selftest for get_thread_arch_aspace_regcache, named
  get_thread_arch_aspace_regcache_test.  To avoid confusion between the
  two functions, rename this one to
  get_thread_arch_aspace_regcache_and_check, I think it describes better
  what it does.

- Remove gdbarch parameter.  We always pass the same gdbarch (the
  current inferior's gdbarch), so having a parameter is not useful.  It
  would be interesting to actually test with multiple gdbarches, to
  verify that the regcache container can hold multiple regcaches (with
  different architectures) for a same (target, ptid).  But it's not the
  case as of this patch.

- Verify that the regcache's arch is correctly set.

- Remove the aspace parameter.  We always pass NULL here, so it's not
  useful to have it as a parameter.  Also, instead of passing a NULL
  aspace to get_thread_arch_aspace_regcache and verifying that we get a
  NULL aspace back, pass the current inferior's aspace (just like we use
  the current inferior's gdbarch).

gdb/ChangeLog:

	* regcache.c (test_get_thread_arch_aspace_regcache): Rename to...
	(get_thread_arch_aspace_regcache_and_check): ... this.  Remove
	gdbarch and aspace parameter.  Use current inferior's aspace.
	Validate regcache's arch value.
	(regcaches_test): Update.

Change-Id: I8b4c2303b4f91f062269043d1f7abe1650232010

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index a6835f0b287f6e78affea1f8b8e34c92bf1426f9..ef36c2fed6869bfa6aa0e0982e727adf7ccac77a 100644 (file)
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,11 @@
+2020-08-20  Simon Marchi  <simon.marchi@polymtl.ca>
+
+       * regcache.c (test_get_thread_arch_aspace_regcache): Rename to...
+       (get_thread_arch_aspace_regcache_and_check): ... this.  Remove
+       gdbarch and aspace parameter.  Use current inferior's aspace.
+       Validate regcache's arch value.
+       (regcaches_test): Update.
+
 2020-08-20  Simon Marchi  <simon.marchi@polymtl.ca>
 
        * regcache.c (regcaches_test): Call registers_changed.

diff --git a/gdb/regcache.c b/gdb/regcache.c
index 9f560acd3edde4bc0980c8a1d0dfab41c492f1d4..c27f6043fb26941729b9700d26c2e124d7ddf76f 100644 (file)
--- a/gdb/regcache.c
+++ b/gdb/regcache.c
@@ -1491,15 +1491,21 @@ regcaches_size ()
 /* Wrapper around get_thread_arch_aspace_regcache that does some self checks.  */
 
 static void
-test_get_thread_arch_aspace_regcache (process_stratum_target *target,
-                                     ptid_t ptid, struct gdbarch *gdbarch,
-                                     address_space *aspace)
+get_thread_arch_aspace_regcache_and_check (process_stratum_target *target,
+                                          ptid_t ptid)
 {
-  struct regcache *regcache
-    = get_thread_arch_aspace_regcache (target, ptid, gdbarch, aspace);
+  /* We currently only test with a single gdbarch.  Any gdbarch will do, so use
+     the current inferior's gdbarch.  Also use the current inferior's address
+     space.  */
+  gdbarch *arch = current_inferior ()->gdbarch;
+  address_space *aspace = current_inferior ()->aspace;
+  regcache *regcache
+    = get_thread_arch_aspace_regcache (target, ptid, arch, aspace);
+
   SELF_CHECK (regcache != NULL);
   SELF_CHECK (regcache->target () == target);
   SELF_CHECK (regcache->ptid () == ptid);
+  SELF_CHECK (regcache->arch () == arch);
   SELF_CHECK (regcache->aspace () == aspace);
 }
 
@@ -1517,37 +1523,27 @@ regcaches_test ()
 
   /* Get regcache from (target1,ptid1), a new regcache is added to
      REGCACHES.  */
-  test_get_thread_arch_aspace_regcache (&test_target1, ptid1,
-                                       target_gdbarch (),
-                                       NULL);
+  get_thread_arch_aspace_regcache_and_check (&test_target1, ptid1);
   SELF_CHECK (regcaches_size () == 1);
 
   /* Get regcache from (target1,ptid2), a new regcache is added to
      REGCACHES.  */
-  test_get_thread_arch_aspace_regcache (&test_target1, ptid2,
-                                       target_gdbarch (),
-                                       NULL);
+  get_thread_arch_aspace_regcache_and_check (&test_target1, ptid2);
   SELF_CHECK (regcaches_size () == 2);
 
   /* Get regcache from (target1,ptid3), a new regcache is added to
      REGCACHES.  */
-  test_get_thread_arch_aspace_regcache (&test_target1, ptid3,
-                                       target_gdbarch (),
-                                       NULL);
+  get_thread_arch_aspace_regcache_and_check (&test_target1, ptid3);
   SELF_CHECK (regcaches_size () == 3);
 
   /* Get regcache from (target1,ptid2) again, nothing is added to
      REGCACHES.  */
-  test_get_thread_arch_aspace_regcache (&test_target1, ptid2,
-                                       target_gdbarch (),
-                                       NULL);
+  get_thread_arch_aspace_regcache_and_check (&test_target1, ptid2);
   SELF_CHECK (regcaches_size () == 3);
 
   /* Get regcache from (target2,ptid2), a new regcache is added to
      REGCACHES, since this time we're using a different target.  */
-  test_get_thread_arch_aspace_regcache (&test_target2, ptid2,
-                                       target_gdbarch (),
-                                       NULL);
+  get_thread_arch_aspace_regcache_and_check (&test_target2, ptid2);
   SELF_CHECK (regcaches_size () == 4);
 
   /* Mark that (target1,ptid2) changed.  The regcache of (target1,
@@ -1557,9 +1553,7 @@ regcaches_test ()
 
   /* Get the regcache from (target2,ptid2) again, confirming the
      registers_changed_ptid call above did not delete it.  */
-  test_get_thread_arch_aspace_regcache (&test_target2, ptid2,
-                                       target_gdbarch (),
-                                       NULL);
+  get_thread_arch_aspace_regcache_and_check (&test_target2, ptid2);
   SELF_CHECK (regcaches_size () == 3);
 
   /* Confirm that marking all regcaches of all targets as changed