Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Simon Marchi via Gdb-patches <gdb-patches@sourceware.org>
To: gdb-patches@sourceware.org
Subject: Re: [PATCH 00/13] DWARF 5 rnglists & loclists fixes (PR 26813)
Date: Tue, 2 Feb 2021 10:43:00 -0500	[thread overview]
Message-ID: <fdc40ba3-24ec-afe8-513a-89846a367021@polymtl.ca> (raw)
In-Reply-To: <20210120053925.142862-1-simon.marchi@polymtl.ca>

I pushed this series.

Simon

On 2021-01-20 12:39 a.m., Simon Marchi wrote:
> This series fixes a few issues related to rnglists and loclists,
> prompted by PR 26813:
> 
>   https://sourceware.org/bugzilla/show_bug.cgi?id=26813
> 
> Simon Marchi (13):
>   gdb/dwarf: change read_loclist_index complaints into errors
>   gdb/dwarf: fix bound check in read_rnglist_index
>   gdb/dwarf: add missing bound check to read_loclist_index
>   gdb/dwarf: remove unnecessary check in read_{rng,loc}list_index
>   gdb/dwarf: few fixes for handling DW_FORM_{rng,loc}listx
>   gdb/dwarf: read correct rnglist/loclist header in
>     read_{rng,loc}list_index
>   gdb/dwarf: read DW_AT_ranges value as unsigned in
>     partial_die_info::read
>   gdb/testsuite: add .debug_rnglists tests
>   gdb/testsuite: DWARF assembler: add context parameters to _location
>   gdb/testsuite: add .debug_loclists tests
>   gdb/dwarf: split dwarf2_cu::ranges_base in two
>   gdb/dwarf: make read_{loc,rng}list_index return sect_offset
>   gdb/testsuite: add test for .debug_{rng,loc}lists section without
>     offset array
> 
>  gdb/dwarf2/attribute.c                        |   5 +-
>  gdb/dwarf2/attribute.h                        |   1 +
>  gdb/dwarf2/die.h                              |  36 +-
>  gdb/dwarf2/read.c                             | 257 ++++++----
>  .../gdb.dwarf2/loclists-multiple-cus.c        |  37 ++
>  .../gdb.dwarf2/loclists-multiple-cus.exp      | 146 ++++++
>  .../gdb.dwarf2/loclists-sec-offset.c          |  69 +++
>  .../gdb.dwarf2/loclists-sec-offset.exp        | 261 ++++++++++
>  .../gdb.dwarf2/rnglists-multiple-cus.exp      | 102 ++++
>  .../gdb.dwarf2/rnglists-sec-offset.exp        | 143 ++++++
>  gdb/testsuite/lib/dwarf.exp                   | 451 +++++++++++++++++-
>  11 files changed, 1381 insertions(+), 127 deletions(-)
>  create mode 100644 gdb/testsuite/gdb.dwarf2/loclists-multiple-cus.c
>  create mode 100644 gdb/testsuite/gdb.dwarf2/loclists-multiple-cus.exp
>  create mode 100644 gdb/testsuite/gdb.dwarf2/loclists-sec-offset.c
>  create mode 100644 gdb/testsuite/gdb.dwarf2/loclists-sec-offset.exp
>  create mode 100644 gdb/testsuite/gdb.dwarf2/rnglists-multiple-cus.exp
>  create mode 100644 gdb/testsuite/gdb.dwarf2/rnglists-sec-offset.exp
> 

      parent reply	other threads:[~2021-02-02 15:43 UTC|newest]

Thread overview: 34+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-01-20  5:39 Simon Marchi via Gdb-patches
2021-01-20  5:39 ` [PATCH 01/13] gdb/dwarf: change read_loclist_index complaints into errors Simon Marchi via Gdb-patches
2021-01-28 15:17   ` Zoran Zaric via Gdb-patches
2021-01-28 15:42     ` Simon Marchi via Gdb-patches
2021-02-25 19:20       ` Tom Tromey
2021-01-20  5:39 ` [PATCH 02/13] gdb/dwarf: fix bound check in read_rnglist_index Simon Marchi via Gdb-patches
2021-01-28 15:22   ` Zoran Zaric via Gdb-patches
2021-01-20  5:39 ` [PATCH 03/13] gdb/dwarf: add missing bound check to read_loclist_index Simon Marchi via Gdb-patches
2021-01-20  5:39 ` [PATCH 04/13] gdb/dwarf: remove unnecessary check in read_{rng, loc}list_index Simon Marchi via Gdb-patches
2021-01-20  5:39 ` [PATCH 05/13] gdb/dwarf: few fixes for handling DW_FORM_{rng, loc}listx Simon Marchi via Gdb-patches
2021-01-28 15:30   ` [PATCH 05/13] gdb/dwarf: few fixes for handling DW_FORM_{rng,loc}listx Zoran Zaric via Gdb-patches
2021-01-20  5:39 ` [PATCH 06/13] gdb/dwarf: read correct rnglist/loclist header in read_{rng, loc}list_index Simon Marchi via Gdb-patches
2021-01-28 15:39   ` [PATCH 06/13] gdb/dwarf: read correct rnglist/loclist header in read_{rng,loc}list_index Zoran Zaric via Gdb-patches
2021-01-28 15:49     ` Simon Marchi via Gdb-patches
2021-01-28 15:54       ` Zoran Zaric via Gdb-patches
2021-01-20  5:39 ` [PATCH 07/13] gdb/dwarf: read DW_AT_ranges value as unsigned in partial_die_info::read Simon Marchi via Gdb-patches
2021-01-28 15:41   ` Zoran Zaric via Gdb-patches
2021-01-28 15:51     ` Simon Marchi via Gdb-patches
2021-01-20  5:39 ` [PATCH 08/13] gdb/testsuite: add .debug_rnglists tests Simon Marchi via Gdb-patches
2021-01-28 16:24   ` Zoran Zaric via Gdb-patches
2021-01-20  5:39 ` [PATCH 09/13] gdb/testsuite: DWARF assembler: add context parameters to _location Simon Marchi via Gdb-patches
2021-01-28 16:30   ` Zoran Zaric via Gdb-patches
2021-01-20  5:39 ` [PATCH 10/13] gdb/testsuite: add .debug_loclists tests Simon Marchi via Gdb-patches
2021-01-28 16:52   ` Zoran Zaric via Gdb-patches
2021-01-28 17:47     ` Simon Marchi via Gdb-patches
2021-01-29 10:13       ` Zoran Zaric via Gdb-patches
2021-01-29 15:57         ` Simon Marchi via Gdb-patches
2021-01-29 16:58           ` Zoran Zaric via Gdb-patches
2021-01-29 17:37             ` Simon Marchi via Gdb-patches
2021-01-20  5:39 ` [PATCH 11/13] gdb/dwarf: split dwarf2_cu::ranges_base in two Simon Marchi via Gdb-patches
2021-01-20  5:39 ` [PATCH 12/13] gdb/dwarf: make read_{loc, rng}list_index return sect_offset Simon Marchi via Gdb-patches
2021-02-25 19:26   ` Tom Tromey
2021-01-20  5:39 ` [PATCH 13/13] gdb/testsuite: add test for .debug_{rng, loc}lists section without offset array Simon Marchi via Gdb-patches
2021-02-02 15:43 ` Simon Marchi via Gdb-patches [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=fdc40ba3-24ec-afe8-513a-89846a367021@polymtl.ca \
    --to=gdb-patches@sourceware.org \
    --cc=simon.marchi@polymtl.ca \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox