[Bug libdw/31111] Handle Clang DWARF 5 DW_AT_decl_file 0

2024-02-12 Thread amerey at redhat dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=3 Aaron Merey changed: What|Removed |Added CC||amerey at redhat dot com Resolu

Re: [PATCH] Handle DW_AT_decl_file 0

2024-02-12 Thread Aaron Merey
Hi Mark, On Mon, Feb 12, 2024 at 4:14 PM Mark Wielaard wrote: > > On Mon, Feb 12, 2024 at 01:16:30PM -0500, Aaron Merey wrote: > > On Mon, Feb 12, 2024 at 12:31 PM Mark Wielaard wrote: > > > >(void) INTUSE(dwarf_getsrclines) (&CUDIE (cu), &lines, &nlines); > > > > - assert (cu->line

Re: [PATCH] Handle DW_AT_decl_file 0

2024-02-12 Thread Mark Wielaard
Hi Aaron, On Mon, Feb 12, 2024 at 01:16:30PM -0500, Aaron Merey wrote: > On Mon, Feb 12, 2024 at 12:31 PM Mark Wielaard wrote: > > >(void) INTUSE(dwarf_getsrclines) (&CUDIE (cu), &lines, &nlines); > > > - assert (cu->lines != NULL); > > > } > > > > I see why would like to get ri

Re: [PATCH] Handle DW_AT_decl_file 0

2024-02-12 Thread Aaron Merey
Hi Mark, On Mon, Feb 12, 2024 at 12:31 PM Mark Wielaard wrote: > >(void) INTUSE(dwarf_getsrclines) (&CUDIE (cu), &lines, &nlines); > > - assert (cu->lines != NULL); > > } > > I see why would like to get rid of asserts in the code base. > But I believe the assert is valid. dwarf_

Re: [PATCH] Handle DW_AT_decl_file 0

2024-02-12 Thread Mark Wielaard
Hi Aaron, On Fri, 2024-02-09 at 21:52 -0500, Aaron Merey wrote: > Modify dwarf_decl_file to support DW_AT_decl_file with value 0. > > Because of inconsistencies in the DWARF 5 spec, it is ambiguous whether > DW_AT_decl_file value 0 is a valid .debug_line file table index for the > main source fil

[PATCH] PR31265 - rework debuginfod archive-extract fdcache

2024-02-12 Thread Frank Ch. Eigler
Hi - I plan to roll out this improvement to all the servers I look after, via COPR builds or such, it's that impactful. commit 7bfc10acc7f0e00c5bc45416d1bf8ee183d79ff3 (HEAD -> master) Author: Frank Ch. Eigler Date: Thu Feb 8 19:33:55 2024 -0500 PR31265 - rework debuginfod archive-extrac