fix calls to find_target_beneath
A long time ago Pedro pointed out that there are some calls to find_target_beneath that pass in an explicit target_ops; but which should instead use the ops provided to the method in question. See: https://sourceware.org/ml/gdb-patches/2014-01/msg00429.html This patch is just a minor cleanup to fix all such calls. There were only three. 2014-07-18 Tom Tromey <tromey@redhat.com> * spu-multiarch.c (spu_region_ok_for_hw_watchpoint): Pass "self" to find_target_beneath. * ravenscar-thread.c (ravenscar_prepare_to_store): Pass "ops" to find_target_beneath. (ravenscar_mourn_inferior): Pass "self" to find_target_beneath.
This commit is contained in:
parent
b0ed115fa5
commit
44e89118cb
|
@ -1,3 +1,11 @@
|
||||||
|
2014-07-18 Tom Tromey <tromey@redhat.com>
|
||||||
|
|
||||||
|
* spu-multiarch.c (spu_region_ok_for_hw_watchpoint): Pass "self"
|
||||||
|
to find_target_beneath.
|
||||||
|
* ravenscar-thread.c (ravenscar_prepare_to_store): Pass "ops" to
|
||||||
|
find_target_beneath.
|
||||||
|
(ravenscar_mourn_inferior): Pass "self" to find_target_beneath.
|
||||||
|
|
||||||
2014-07-18 Tom Tromey <tromey@redhat.com>
|
2014-07-18 Tom Tromey <tromey@redhat.com>
|
||||||
|
|
||||||
PR gdb/17130:
|
PR gdb/17130:
|
||||||
|
|
|
@ -308,7 +308,7 @@ static void
|
||||||
ravenscar_prepare_to_store (struct target_ops *self,
|
ravenscar_prepare_to_store (struct target_ops *self,
|
||||||
struct regcache *regcache)
|
struct regcache *regcache)
|
||||||
{
|
{
|
||||||
struct target_ops *beneath = find_target_beneath (&ravenscar_ops);
|
struct target_ops *beneath = find_target_beneath (self);
|
||||||
|
|
||||||
if (!ravenscar_runtime_initialized ()
|
if (!ravenscar_runtime_initialized ()
|
||||||
|| ptid_equal (inferior_ptid, base_magic_null_ptid)
|
|| ptid_equal (inferior_ptid, base_magic_null_ptid)
|
||||||
|
@ -327,7 +327,7 @@ ravenscar_prepare_to_store (struct target_ops *self,
|
||||||
static void
|
static void
|
||||||
ravenscar_mourn_inferior (struct target_ops *ops)
|
ravenscar_mourn_inferior (struct target_ops *ops)
|
||||||
{
|
{
|
||||||
struct target_ops *beneath = find_target_beneath (&ravenscar_ops);
|
struct target_ops *beneath = find_target_beneath (ops);
|
||||||
|
|
||||||
base_ptid = null_ptid;
|
base_ptid = null_ptid;
|
||||||
beneath->to_mourn_inferior (beneath);
|
beneath->to_mourn_inferior (beneath);
|
||||||
|
|
|
@ -121,7 +121,7 @@ static int
|
||||||
spu_region_ok_for_hw_watchpoint (struct target_ops *self,
|
spu_region_ok_for_hw_watchpoint (struct target_ops *self,
|
||||||
CORE_ADDR addr, int len)
|
CORE_ADDR addr, int len)
|
||||||
{
|
{
|
||||||
struct target_ops *ops_beneath = find_target_beneath (&spu_ops);
|
struct target_ops *ops_beneath = find_target_beneath (self);
|
||||||
|
|
||||||
/* We cannot watch SPU local store. */
|
/* We cannot watch SPU local store. */
|
||||||
if (SPUADDR_SPU (addr) != -1)
|
if (SPUADDR_SPU (addr) != -1)
|
||||||
|
|
Loading…
Reference in New Issue