diff options
author | Tom Tromey <tom@tromey.com> | 2021-03-08 07:27:57 -0700 |
---|---|---|
committer | Tom Tromey <tom@tromey.com> | 2021-03-08 07:28:05 -0700 |
commit | aec95807f11b3403c1e8e000618bc85e0440799e (patch) | |
tree | ed4caafc97f6ed99fd37c6cf5c1738c3f31ddc08 /gdb/eval.c | |
parent | a220ead5d47c7a026fa164004390fc477df63ebe (diff) | |
download | gdb-aec95807f11b3403c1e8e000618bc85e0440799e.zip gdb-aec95807f11b3403c1e8e000618bc85e0440799e.tar.gz gdb-aec95807f11b3403c1e8e000618bc85e0440799e.tar.bz2 |
Split out eval_op_type
This splits OP_TYPE into a new function for future use.
gdb/ChangeLog
2021-03-08 Tom Tromey <tom@tromey.com>
* eval.c (eval_op_type): New function.
(evaluate_subexp_standard): Use it.
Diffstat (limited to 'gdb/eval.c')
-rw-r--r-- | gdb/eval.c | 21 |
1 files changed, 15 insertions, 6 deletions
@@ -2039,6 +2039,20 @@ eval_op_postdec (struct type *expect_type, struct expression *exp, } } +/* A helper function for OP_TYPE. */ + +static struct value * +eval_op_type (struct type *expect_type, struct expression *exp, + enum noside noside, struct type *type) +{ + if (noside == EVAL_SKIP) + return eval_skip_value (exp); + else if (noside == EVAL_AVOID_SIDE_EFFECTS) + return allocate_value (type); + else + error (_("Attempt to use a type name as an expression")); +} + struct value * evaluate_subexp_standard (struct type *expect_type, struct expression *exp, int *pos, @@ -2994,12 +3008,7 @@ evaluate_subexp_standard (struct type *expect_type, /* The value is not supposed to be used. This is here to make it easier to accommodate expressions that contain types. */ (*pos) += 2; - if (noside == EVAL_SKIP) - return eval_skip_value (exp); - else if (noside == EVAL_AVOID_SIDE_EFFECTS) - return allocate_value (exp->elts[pc + 1].type); - else - error (_("Attempt to use a type name as an expression")); + return eval_op_type (expect_type, exp, noside, exp->elts[pc + 1].type); case OP_TYPEOF: case OP_DECLTYPE: |