diff options
| author | Iain Buclaw <ibuclaw@gdcproject.org> | 2017-01-08 11:17:54 +0100 |
|---|---|---|
| committer | Iain Buclaw <ibuclaw@gdcproject.org> | 2017-01-08 11:20:56 +0100 |
| commit | f5e6296e2194add209b546ad49039753a10242f5 (patch) | |
| tree | 2047bd675aee2103a1a689db5b8cc34c1606df22 /gdb/d-exp.y | |
| parent | 2efa21564b81bd6d8a7ee23c2868a4bd6749b11c (diff) | |
| download | binutils-gdb-f5e6296e2194add209b546ad49039753a10242f5.tar.gz | |
[D] Fix crash when debug expression enabled.
While casting works as expected with expression debugging turned off,
this seems to be an indication that the D language parser function is
doing something wrong in the building of the expression.
Without changing the grammar, using UNOP_CAST_TYPE is the right thing to
do here, as the TypeExp handler has already wrapped the type around a
pair of OP_TYPE opcodes.
gdb/ChangeLog:
* d-exp.y (CastExpression): Emit UNOP_CAST_TYPE.
gdb/testsuite/ChangeLog:
* gdb.dlang/debug-expr.exp: New file.
Diffstat (limited to 'gdb/d-exp.y')
| -rw-r--r-- | gdb/d-exp.y | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/gdb/d-exp.y b/gdb/d-exp.y index 077e645dec2..b5265754ed0 100644 --- a/gdb/d-exp.y +++ b/gdb/d-exp.y @@ -321,15 +321,12 @@ UnaryExpression: CastExpression: CAST_KEYWORD '(' TypeExp ')' UnaryExpression - { write_exp_elt_opcode (pstate, UNOP_CAST); - write_exp_elt_type (pstate, $3); - write_exp_elt_opcode (pstate, UNOP_CAST); } + { write_exp_elt_opcode (pstate, UNOP_CAST_TYPE); } /* C style cast is illegal D, but is still recognised in the grammar, so we keep this around for convenience. */ | '(' TypeExp ')' UnaryExpression - { write_exp_elt_opcode (pstate, UNOP_CAST); - write_exp_elt_type (pstate, $2); - write_exp_elt_opcode (pstate, UNOP_CAST); } + { write_exp_elt_opcode (pstate, UNOP_CAST_TYPE); } + ; PowExpression: |