[binutils-gdb] Convert exp_uses_objfile to a method of expression

[binutils-gdb] Convert exp_uses_objfile to a method of expression

[Tom Tromey]

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

commit aa9bd4452873136e7406f68fc51e66ef5951190b
Author: Tom Tromey <tromey@adacore.com>
Date:   Mon Dec 19 11:14:02 2022 -0700

    Convert exp_uses_objfile to a method of expression
    
    This changes the exp_uses_objfile function to be a method of
    'expression'.
    
    Reviewed-By: Lancelot Six <lancelot.six@amd.com>

Diff:
---
 gdb/eval.c        |  9 +++++++++
 gdb/expression.h  |  5 +++++
 gdb/parse.c       | 12 ------------
 gdb/parser-defs.h |  2 --
 gdb/printcmd.c    |  2 +-
 gdb/varobj.c      |  3 +--
 6 files changed, 16 insertions(+), 17 deletions(-)

diff --git a/gdb/eval.c b/gdb/eval.c
index 191c680a87b..05e35941101 100644
--- a/gdb/eval.c
+++ b/gdb/eval.c
@@ -89,6 +89,15 @@ parse_to_comma_and_eval (const char **expp)
 
 /* See expression.h.  */
 
+bool
+expression::uses_objfile (struct objfile *objfile) const
+{
+  gdb_assert (objfile->separate_debug_objfile_backlink == nullptr);
+  return op->uses_objfile (objfile);
+}
+
+/* See expression.h.  */
+
 struct value *
 expression::evaluate (struct type *expect_type, enum noside noside)
 {
diff --git a/gdb/expression.h b/gdb/expression.h
index 2e00da90509..c314fc24e1b 100644
--- a/gdb/expression.h
+++ b/gdb/expression.h
@@ -214,6 +214,11 @@ struct expression
     op->dump (stream, 0);
   }
 
+  /* Return true if this expression uses OBJFILE (and will become
+     dangling when OBJFILE is unloaded), otherwise return false.
+     OBJFILE must not be a separate debug info file.  */
+  bool uses_objfile (struct objfile *objfile) const;
+
   /* Evaluate the expression.  EXPECT_TYPE is the context type of the
      expression; normally this should be nullptr.  NOSIDE controls how
      evaluation is performed.  */
diff --git a/gdb/parse.c b/gdb/parse.c
index 29d7bf172f8..2f7d58061ab 100644
--- a/gdb/parse.c
+++ b/gdb/parse.c
@@ -667,18 +667,6 @@ parser_fprintf (FILE *x, const char *y, ...)
   va_end (args);
 }
 
-/* Return rue if EXP uses OBJFILE (and will become dangling when
-   OBJFILE is unloaded), otherwise return false.  OBJFILE must not be
-   a separate debug info file.  */
-
-bool
-exp_uses_objfile (struct expression *exp, struct objfile *objfile)
-{
-  gdb_assert (objfile->separate_debug_objfile_backlink == NULL);
-
-  return exp->op->uses_objfile (objfile);
-}
-
 void _initialize_parse ();
 void
 _initialize_parse ()
diff --git a/gdb/parser-defs.h b/gdb/parser-defs.h
index 3b69deeead5..1780d856744 100644
--- a/gdb/parser-defs.h
+++ b/gdb/parser-defs.h
@@ -421,7 +421,5 @@ extern bool fits_in_type (int n_sign, ULONGEST n, int type_bits,
 
 extern void parser_fprintf (FILE *, const char *, ...) ATTRIBUTE_PRINTF (2, 3);
 
-extern bool exp_uses_objfile (struct expression *exp, struct objfile *objfile);
-
 #endif /* PARSER_DEFS_H */
 
diff --git a/gdb/printcmd.c b/gdb/printcmd.c
index 67a34009c13..13b979c3720 100644
--- a/gdb/printcmd.c
+++ b/gdb/printcmd.c
@@ -2360,7 +2360,7 @@ clear_dangling_display_expressions (struct objfile *objfile)
 	}
 
       if (bl_objf == objfile
-	  || (d->exp != NULL && exp_uses_objfile (d->exp.get (), objfile)))
+	  || (d->exp != nullptr && d->exp->uses_objfile (objfile)))
 	{
 	  d->exp.reset ();
 	  d->block = NULL;
diff --git a/gdb/varobj.c b/gdb/varobj.c
index 8470600d19e..eb47ecada3e 100644
--- a/gdb/varobj.c
+++ b/gdb/varobj.c
@@ -2412,8 +2412,7 @@ varobj_invalidate_if_uses_objfile (struct objfile *objfile)
 	    }
 	}
 
-      if (var->root->exp != nullptr
-	  && exp_uses_objfile (var->root->exp.get (), objfile))
+      if (var->root->exp != nullptr && var->root->exp->uses_objfile (objfile))
 	{
 	  /* The varobj's current expression references the objfile.  For
 	     globals and floating, it is possible that when we try to