From: Tom Tromey <tom@tromey.com>
Date: Sat, 9 Jan 2021 17:06:25 +0000 (-0700)
Subject: Remove a use of print_expression
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=bc167b6b3eb81de168c54aed4a8f0d1fdaa065e2;p=binutils-gdb.git

Remove a use of print_expression

The tracepoint code uses print_expression to reconstruct an expression
string.  However, the original expression is already available -- it
was just parsed a bit earlier in the same function.  This patch
changes this code to simply save the already-parsed expression, rather
than attempt to reconstruct it.

gdb/ChangeLog
2021-01-09  Tom Tromey  <tom@tromey.com>

	* tracepoint.h (class collection_list) <append_exp>: Take a
	std::string.
	* tracepoint.c (collection_list::append_exp): Take a std::string.
	(encode_actions_1): Update.
---

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index d567830b1fa..2751dfe64a0 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,10 @@
+2021-01-09  Tom Tromey  <tom@tromey.com>
+
+	* tracepoint.h (class collection_list) <append_exp>: Take a
+	std::string.
+	* tracepoint.c (collection_list::append_exp): Take a std::string.
+	(encode_actions_1): Update.
+
 2021-01-08  Tom Tromey  <tromey@adacore.com>
 
 	* parse.c (parse_expression): Add void_context_p parameter.  Use
diff --git a/gdb/tracepoint.c b/gdb/tracepoint.c
index 7109fd8cb9d..17d70a6a8d4 100644
--- a/gdb/tracepoint.c
+++ b/gdb/tracepoint.c
@@ -1266,16 +1266,12 @@ collection_list::stringify ()
   return str_list;
 }
 
-/* Add the printed expression EXP to *LIST.  */
+/* Add the expression STR to M_COMPUTED.  */
 
 void
-collection_list::append_exp (struct expression *exp)
+collection_list::append_exp (std::string &&str)
 {
-  string_file tmp_stream;
-
-  print_expression (exp, &tmp_stream);
-
-  m_computed.push_back (std::move (tmp_stream.string ()));
+  m_computed.push_back (std::move (str));
 }
 
 void
@@ -1379,6 +1375,7 @@ encode_actions_1 (struct command_line *action,
 		{
 		  unsigned long addr;
 
+		  const char *exp_start = action_exp;
 		  expression_up exp = parse_exp_1 (&action_exp, tloc->address,
 						   block_for_pc (tloc->address),
 						   1);
@@ -1412,7 +1409,8 @@ encode_actions_1 (struct command_line *action,
 					     memrange_absolute, addr,
 					     TYPE_LENGTH (exp->elts[1].type),
 					     tloc->address);
-		      collect->append_exp (exp.get ());
+		      collect->append_exp (std::string (exp_start,
+							action_exp));
 		      break;
 
 		    case OP_VAR_VALUE:
@@ -1441,7 +1439,8 @@ encode_actions_1 (struct command_line *action,
 		      collect->add_ax_registers (aexpr.get ());
 
 		      collect->add_aexpr (std::move (aexpr));
-		      collect->append_exp (exp.get ());
+		      collect->append_exp (std::string (exp_start,
+							action_exp));
 		      break;
 		    }		/* switch */
 		}		/* do */
diff --git a/gdb/tracepoint.h b/gdb/tracepoint.h
index 294f011fe95..b32dd61fdfc 100644
--- a/gdb/tracepoint.h
+++ b/gdb/tracepoint.h
@@ -257,7 +257,7 @@ public:
 
   void add_wholly_collected (const char *print_name);
 
-  void append_exp (struct expression *exp);
+  void append_exp (std::string &&exp);
 
   /* Add AEXPR to the list, taking ownership.  */
   void add_aexpr (agent_expr_up aexpr);