espressif/binutils-gdb
1
0
Fork 0
mirror of https://github.com/espressif/binutils-gdb.git synced 2025-10-16 12:24:19 +08:00
Code Issues Packages Projects Releases Wiki Activity

Warn for missing separate debug files only if needed

Browse Source 
We shouldn't warn missing separate debug files when debug info isn't
needed.

	PR binutils/27486
	* dwarf.c (load_separate_debug_info): Issue warning only if
	do_debug_links is set.
	* testsuite/binutils-all/compress.exp: Run objdump and readelf
	with missing debug file.
This commit is contained in:
H.J. Lu
2021-03-01 05:34:34 -08:00
parent 8ee10e8609
commit ecd65684f5
3 changed files with 54 additions and 23 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
binutils/ChangeLog
View File

@ -1,3 +1,11 @@
2021-03-01 H.J. Lu <hongjiu.lu@intel.com>
PR binutils/27486
* dwarf.c (load_separate_debug_info): Issue warning only if
do_debug_links is set.
* testsuite/binutils-all/compress.exp: Run objdump and readelf
with missing debug file.
2021-03-01 Alan Modra <amodra@gmail.com>
PR 27128

18
binutils/dwarf.c
View File

@ -11210,24 +11210,31 @@ load_separate_debug_info (const char * main_filename,
}
#endif
if (do_debug_links)
{
/* Failed to find the file. */
warn (_("could not find separate debug file '%s'\n"), separate_filename);
warn (_("could not find separate debug file '%s'\n"),
separate_filename);
warn (_("tried: %s\n"), debug_filename);
#ifdef EXTRA_DEBUG_ROOT2
sprintf (debug_filename, "%s/%s", EXTRA_DEBUG_ROOT2, separate_filename);
sprintf (debug_filename, "%s/%s", EXTRA_DEBUG_ROOT2,
separate_filename);
warn (_("tried: %s\n"), debug_filename);
#endif
#ifdef EXTRA_DEBUG_ROOT1
sprintf (debug_filename, "%s/%s/%s", EXTRA_DEBUG_ROOT1, canon_dir, separate_filename);
sprintf (debug_filename, "%s/%s/%s", EXTRA_DEBUG_ROOT1,
canon_dir, separate_filename);
warn (_("tried: %s\n"), debug_filename);
sprintf (debug_filename, "%s/%s", EXTRA_DEBUG_ROOT1, separate_filename);
sprintf (debug_filename, "%s/%s", EXTRA_DEBUG_ROOT1,
separate_filename);
warn (_("tried: %s\n"), debug_filename);
#endif
sprintf (debug_filename, "%s.debug/%s", canon_dir, separate_filename);
sprintf (debug_filename, "%s.debug/%s", canon_dir,
separate_filename);
warn (_("tried: %s\n"), debug_filename);
sprintf (debug_filename, "%s%s", canon_dir, separate_filename);
@ -11248,6 +11255,7 @@ load_separate_debug_info (const char * main_filename,
warn (_("tried: DEBUGINFOD_URLS=%s\n"), urls);
}
#endif
}
free (canon_dir);
free (debug_filename);

15
binutils/testsuite/binutils-all/compress.exp
View File

@ -676,6 +676,7 @@ proc test_gnu_debuglink {} {
global STRIP
global OBJCOPY
global OBJDUMP
global READELF
global gcc_gas_flag
set test "gnu-debuglink"
@ -757,6 +758,20 @@ proc test_gnu_debuglink {} {
} else {
pass "$test (objdump 2)"
}
remote_file host delete tmpdir/testprog.compress
set got [remote_exec host [concat sh -c [list "$OBJDUMP --syms tmpdir/testprog > /dev/null"]]]
if { [lindex $got 0] != 0 || ![string match "" [lindex $got 1]] } then {
fail "$test (objdump with missing link)"
} else {
pass "$test (objdump with missing link)"
}
set got [remote_exec host [concat sh -c [list "$READELF -S tmpdir/testprog > /dev/null"]]]
if { [lindex $got 0] != 0 || ![string match "" [lindex $got 1]] } then {
fail "$test (readelf with missing link)"
} else {
pass "$test (readelf with missing link)"
}
}
if {[is_elf_format]} then {