diff options
author | Alan Modra <amodra@gmail.com> | 2022-03-08 22:48:51 +1030 |
---|---|---|
committer | Alan Modra <amodra@gmail.com> | 2022-03-09 15:50:34 +1030 |
commit | fb0e49d8e05e61ca2af9b5f60b01ad5fb6d274ff (patch) | |
tree | be87267c5f39fc737768b36ecbe68d78351ca2d2 /gas/dwarf2dbg.c | |
parent | 4afc8894393b31dae6845ef1f6e4cddf9ef5b165 (diff) | |
download | gdb-fb0e49d8e05e61ca2af9b5f60b01ad5fb6d274ff.zip gdb-fb0e49d8e05e61ca2af9b5f60b01ad5fb6d274ff.tar.gz gdb-fb0e49d8e05e61ca2af9b5f60b01ad5fb6d274ff.tar.bz2 |
Constant fold view increment expressions
The idea here is to replace expressions like v + 1 + 1 + 1 with v + 3.
* dwarf2dbg.c (set_or_check_view): Remove useless assertion.
Resolve multiple view increments.
* testsuite/gas/elf/dwarf2-18.d: Don't xfail mep.
Diffstat (limited to 'gas/dwarf2dbg.c')
-rw-r--r-- | gas/dwarf2dbg.c | 17 |
1 files changed, 13 insertions, 4 deletions
diff --git a/gas/dwarf2dbg.c b/gas/dwarf2dbg.c index 7269c4c..ba97b5f 100644 --- a/gas/dwarf2dbg.c +++ b/gas/dwarf2dbg.c @@ -402,18 +402,27 @@ set_or_check_view (struct line_entry *e, struct line_entry *p, if (viewx.X_op != O_constant || viewx.X_add_number) { expressionS incv; + expressionS *p_view; if (!p->loc.u.view) - { - p->loc.u.view = symbol_temp_make (); - gas_assert (!S_IS_DEFINED (p->loc.u.view)); - } + p->loc.u.view = symbol_temp_make (); memset (&incv, 0, sizeof (incv)); incv.X_unsigned = 1; incv.X_op = O_symbol; incv.X_add_symbol = p->loc.u.view; incv.X_add_number = 1; + p_view = symbol_get_value_expression (p->loc.u.view); + if (p_view->X_op == O_constant || p_view->X_op == O_symbol) + { + /* If we can, constant fold increments so that a chain of + expressions v + 1 + 1 ... + 1 is not created. + resolve_expression isn't ideal for this purpose. The + base v might not be resolvable until later. */ + incv.X_op = p_view->X_op; + incv.X_add_symbol = p_view->X_add_symbol; + incv.X_add_number = p_view->X_add_number + 1; + } if (viewx.X_op == O_constant) { |