From c4f5871457222f2c4a99e662dfa16b7c662f750a Mon Sep 17 00:00:00 2001 From: Hans-Peter Nilsson Date: Wed, 12 Jan 2022 05:41:09 +0100 Subject: objdump, readelf: Emit "CU:" format only when wide output is requested As pre-approved by Alan in https://sourceware.org/pipermail/binutils/2021-September/118019.html and I believe people have run into getting testsuite failures for test-environments with "long" directory names, at least once more since that time. Enough. I grepped the gas, binutils and ld testsuites for "CU:" to catch target-specific occurrences, but I noticed none. I chose to remove "CU:" on the objdump tests instead of changing options to get the wide format, so as to keep the name of the test consistent with actual options; but added it to the readelf options for the gas test as I believe the "CU:" format is preferable. Tested for cris-elf and native x86_64-pc-linux-gnu. binutils: * dwarf.c (display_debug_lines_decoded): Don't check the string length of the directory, instead emit the "CU: dir/name" format only if wide output is requested. * testsuite/binutils-all/dw5.W, testsuite/binutils-all/objdump.WL: Adjust accordingly. gas: * testsuite/gas/elf/dwarf-5-loc0.d: Add -W to readelf options. --- binutils/dwarf.c | 2 +- binutils/testsuite/binutils-all/dw5.W | 2 +- binutils/testsuite/binutils-all/objdump.WL | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) (limited to 'binutils') diff --git a/binutils/dwarf.c b/binutils/dwarf.c index 52d4dd1..bc33e9f 100644 --- a/binutils/dwarf.c +++ b/binutils/dwarf.c @@ -5137,7 +5137,7 @@ display_debug_lines_decoded (struct dwarf_section * section, else directory = (char *) directory_table[ix - 1]; - if (do_wide || strlen (directory) < 76) + if (do_wide) printf (_("CU: %s/%s:\n"), directory, file_table[0].name); else printf ("%s:\n", file_table[0].name); diff --git a/binutils/testsuite/binutils-all/dw5.W b/binutils/testsuite/binutils-all/dw5.W index 562b64f..ebab8b7 100644 --- a/binutils/testsuite/binutils-all/dw5.W +++ b/binutils/testsuite/binutils-all/dw5.W @@ -347,7 +347,7 @@ Raw dump of debug contents of section .debug_line: Contents of the .debug_line section: -CU: ./main.c: +main.c: File name Line number Starting address View Stmt main.c 6 0x1234 x main.c 6 0x12346 x diff --git a/binutils/testsuite/binutils-all/objdump.WL b/binutils/testsuite/binutils-all/objdump.WL index e16d5ba..f02a9e6 100644 --- a/binutils/testsuite/binutils-all/objdump.WL +++ b/binutils/testsuite/binutils-all/objdump.WL @@ -3,7 +3,7 @@ Contents of the \.debug_line section: -CU: \./dw2-decodedline\.c: +dw2-decodedline\.c: File name Line number Starting address View Stmt directory/file1\.c: -- cgit v1.1