[PATCH v2] dwarf: Dump .debug_loclists only for DWARF-5

[Jan Beulich]

On 24.03.2025 17:18, H.J. Lu wrote:
> .debug_loclists section is loaded into debug_information as DWARF-5 debug
> info and .debug_loc section is loaded into debug_information as pre-DWARF-5
> debug info.  When dumping .debug_loc section, we should only process
> pre-DWARF-5 debug info in debug_information.  When dumping .debug_loclists
> section, we should only process DWARF-5 info in debug_information.
> 
> binutils/
> 
> 	PR binutils/32809
> 	* dwarf.c (display_debug_loc): Dump .debug_loclists only for
> 	DWARF-5.
> 
> ld/
> 
> 	PR binutils/32809
> 	* testsuite/ld-x86-64/dwarf4.s: New file.
> 	* testsuite/ld-x86-64/dwarf5a.s: Likewise.
> 	* testsuite/ld-x86-64/dwarf5b.s: Likewise.
> 	* testsuite/ld-x86-64/pr32809.d: Likewise.
> 	* testsuite/ld-x86-64/x86-64.exp: Run pr32809.
> 
> Signed-off-by: H.J. Lu <hjl.tools@gmail.com>

Okay. I'd like to mention though that I dislike the kind of testcases you add
here. How in the world is someone supposed to be able to reasonably quickly
establish what the source files are supposed to be doing, when they're
(apparently) bare, un-commented compiler generated code? And not even stripped
of anything that's obviously unrelated to the tests (like .ident).

Jan