summaryrefslogtreecommitdiff
path: root/gdb/expop.h
diff options
context:
space:
mode:
authorTom Tromey <tromey@adacore.com>2022-03-09 14:35:10 -0700
committerTom Tromey <tromey@adacore.com>2022-03-16 09:28:13 -0600
commita73c128df6e149c36940240f2b2198f9732ab6c7 (patch)
tree36727e90b541814d575cda666114b5c1aec404a8 /gdb/expop.h
parentfc18a21b65a7a0fc05d5e41118e072d253aeca0a (diff)
downloadbinutils-gdb-a73c128df6e149c36940240f2b2198f9732ab6c7.tar.gz
Remove eval_op_concat
eval_op_concat has code to search for an operator overload of BINOP_CONCAT. However, the operator overloading code is specific to C++, which does not have this operator. And, binop_types_user_defined_p rejects this case right at the start, and value_x_binop does not handle this case. I think this code has been dead for a very long time. This patch removes it and hoists the remaining call into concatenation::evaluate, removing eval_op_concat entirely.
Diffstat (limited to 'gdb/expop.h')
-rw-r--r--gdb/expop.h6
1 files changed, 1 insertions, 5 deletions
diff --git a/gdb/expop.h b/gdb/expop.h
index 1592568a857..d903ab0bb7e 100644
--- a/gdb/expop.h
+++ b/gdb/expop.h
@@ -83,10 +83,6 @@ extern struct value *eval_op_member (struct type *expect_type,
struct expression *exp,
enum noside noside,
struct value *arg1, struct value *arg2);
-extern struct value *eval_op_concat (struct type *expect_type,
- struct expression *exp,
- enum noside noside,
- struct value *arg1, struct value *arg2);
extern struct value *eval_op_add (struct type *expect_type,
struct expression *exp,
enum noside noside,
@@ -1158,7 +1154,7 @@ public:
= std::get<0> (m_storage)->evaluate_with_coercion (exp, noside);
value *rhs
= std::get<1> (m_storage)->evaluate_with_coercion (exp, noside);
- return eval_op_concat (expect_type, exp, noside, lhs, rhs);
+ return value_concat (lhs, rhs);
}
enum exp_opcode opcode () const override