espressif/binutils-gdb
1
0
Fork 0
mirror of https://github.com/espressif/binutils-gdb.git synced 2025-07-01 09:49:43 +08:00
Code Issues Packages Projects Releases Wiki Activity

[gdb] Fix s390x -m31 build

Browse Source 
When building gdb on s390x with -m31, we run into this Wformat
warning (which Werror turns into an error):
...
gdb/dwarf2read.c: In function \
  'void create_addrmap_from_aranges(dwarf2_per_objfile*, \
                                    dwarf2_section_info*)':
gdb/dwarf2read.c:3277:22: error: format '%zu' expects argument of type \
  'size_t', but argument 3 has type 'int' [-Werror=format=]
    warning (_("Section .debug_aranges in %s entry at offset %zu "
...

The Wformat warning is triggered in this statement:
...
          warning (_("Section .debug_aranges in %s entry at offset %zu "
                     "length %s exceeds section length %s, "
                     "ignoring .debug_aranges."),
                   objfile_name (objfile), entry_addr - section->buffer,
                   plongest (bytes_read + entry_length),
                   pulongest (section->size));
...
where 'entry_addr - section->buffer' is of type ptrdiff_t and '%zu' prints an
unsigned with the same size as size_t/ssize_t.

On s390x with -m31, we have:
- size_t   : unsigned long int (32-bit)
- ptrdiff_t: int               (32-bit)

Wformat warns against this because even though long int and int have the same
size, the types are not compatible.

[ The Wformat warning is to similar to what we would get for x86_64 -m32
(where long and int are also the same size) and:
...
int i;
printf ("%ld", i);
... ]

Fix this by using '%s' and plongest instead of '%zu' to print ptrdiff_t.

Build and reg-tested on x86_64.

gdb/ChangeLog:

2019-06-22  Tom de Vries  <tdevries@suse.de>

	* dwarf2read.c (create_addrmap_from_aranges)
	(read_debug_names_from_section): Print ptrdiff_t using '%s' and plongest
	instead of '%zu'.
This commit is contained in:
Tom de Vries
2019-06-22 10:59:37 +02:00
parent 5ec331ef74
commit 47e3f47487
2 changed files with 30 additions and 18 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
gdb/ChangeLog
View File

@ -1,3 +1,9 @@
2019-06-22 Tom de Vries <tdevries@suse.de>
* dwarf2read.c (create_addrmap_from_aranges)
(read_debug_names_from_section): Print ptrdiff_t using '%s' and plongest
instead of '%zu'.
2019-06-21 Simon Marchi <simon.marchi@efficios.com> 2019-06-21 Simon Marchi <simon.marchi@efficios.com>
* dwarf2read.h (dwarf2_section_info_def): Remove. * dwarf2read.h (dwarf2_section_info_def): Remove.

42
gdb/dwarf2read.c
View File

@ -3229,10 +3229,11 @@ create_addrmap_from_aranges (struct dwarf2_per_objfile *dwarf2_per_objfile,
const uint8_t offset_size = dwarf5_is_dwarf64 ? 8 : 4; const uint8_t offset_size = dwarf5_is_dwarf64 ? 8 : 4;
if (addr + entry_length > section->buffer + section->size) if (addr + entry_length > section->buffer + section->size)
{ {
warning (_("Section .debug_aranges in %s entry at offset %zu " warning (_("Section .debug_aranges in %s entry at offset %s "
"length %s exceeds section length %s, " "length %s exceeds section length %s, "
"ignoring .debug_aranges."), "ignoring .debug_aranges."),
objfile_name (objfile), entry_addr - section->buffer, objfile_name (objfile),
plongest (entry_addr - section->buffer),
plongest (bytes_read + entry_length), plongest (bytes_read + entry_length),
pulongest (section->size)); pulongest (section->size));
return; return;
@ -3243,10 +3244,10 @@ create_addrmap_from_aranges (struct dwarf2_per_objfile *dwarf2_per_objfile,
addr += 2; addr += 2;
if (version != 2) if (version != 2)
{ {
warning (_("Section .debug_aranges in %s entry at offset %zu " warning (_("Section .debug_aranges in %s entry at offset %s "
"has unsupported version %d, ignoring .debug_aranges."), "has unsupported version %d, ignoring .debug_aranges."),
objfile_name (objfile), entry_addr - section->buffer, objfile_name (objfile),
version); plongest (entry_addr - section->buffer), version);
return; return;
} }
@ -3257,10 +3258,11 @@ create_addrmap_from_aranges (struct dwarf2_per_objfile *dwarf2_per_objfile,
= debug_info_offset_to_per_cu.find (sect_offset (debug_info_offset)); = debug_info_offset_to_per_cu.find (sect_offset (debug_info_offset));
if (per_cu_it == debug_info_offset_to_per_cu.cend ()) if (per_cu_it == debug_info_offset_to_per_cu.cend ())
{ {
warning (_("Section .debug_aranges in %s entry at offset %zu " warning (_("Section .debug_aranges in %s entry at offset %s "
"debug_info_offset %s does not exists, " "debug_info_offset %s does not exists, "
"ignoring .debug_aranges."), "ignoring .debug_aranges."),
objfile_name (objfile), entry_addr - section->buffer, objfile_name (objfile),
plongest (entry_addr - section->buffer),
pulongest (debug_info_offset)); pulongest (debug_info_offset));
return; return;
} }
@ -3269,20 +3271,21 @@ create_addrmap_from_aranges (struct dwarf2_per_objfile *dwarf2_per_objfile,
const uint8_t address_size = *addr++; const uint8_t address_size = *addr++;
if (address_size < 1 || address_size > 8) if (address_size < 1 || address_size > 8)
{ {
warning (_("Section .debug_aranges in %s entry at offset %zu " warning (_("Section .debug_aranges in %s entry at offset %s "
"address_size %u is invalid, ignoring .debug_aranges."), "address_size %u is invalid, ignoring .debug_aranges."),
objfile_name (objfile), entry_addr - section->buffer, objfile_name (objfile),
address_size); plongest (entry_addr - section->buffer), address_size);
return; return;
} }
const uint8_t segment_selector_size = *addr++; const uint8_t segment_selector_size = *addr++;
if (segment_selector_size != 0) if (segment_selector_size != 0)
{ {
warning (_("Section .debug_aranges in %s entry at offset %zu " warning (_("Section .debug_aranges in %s entry at offset %s "
"segment_selector_size %u is not supported, " "segment_selector_size %u is not supported, "
"ignoring .debug_aranges."), "ignoring .debug_aranges."),
objfile_name (objfile), entry_addr - section->buffer, objfile_name (objfile),
plongest (entry_addr - section->buffer),
segment_selector_size); segment_selector_size);
return; return;
} }
@ -3295,9 +3298,10 @@ create_addrmap_from_aranges (struct dwarf2_per_objfile *dwarf2_per_objfile,
padding > 0; padding--) padding > 0; padding--)
if (*addr++ != 0) if (*addr++ != 0)
{ {
warning (_("Section .debug_aranges in %s entry at offset %zu " warning (_("Section .debug_aranges in %s entry at offset %s "
"padding is not zero, ignoring .debug_aranges."), "padding is not zero, ignoring .debug_aranges."),
objfile_name (objfile), entry_addr - section->buffer); objfile_name (objfile),
plongest (entry_addr - section->buffer));
return; return;
} }
@ -3305,10 +3309,11 @@ create_addrmap_from_aranges (struct dwarf2_per_objfile *dwarf2_per_objfile,
{ {
if (addr + 2 * address_size > entry_end) if (addr + 2 * address_size > entry_end)
{ {
warning (_("Section .debug_aranges in %s entry at offset %zu " warning (_("Section .debug_aranges in %s entry at offset %s "
"address list is not properly terminated, " "address list is not properly terminated, "
"ignoring .debug_aranges."), "ignoring .debug_aranges."),
objfile_name (objfile), entry_addr - section->buffer); objfile_name (objfile),
plongest (entry_addr - section->buffer));
return; return;
} }
ULONGEST start = extract_unsigned_integer (addr, address_size, ULONGEST start = extract_unsigned_integer (addr, address_size,
@ -5517,8 +5522,9 @@ read_debug_names_from_section (struct objfile *objfile,
if (addr != abbrev_table_start + abbrev_table_size) if (addr != abbrev_table_start + abbrev_table_size)
{ {
warning (_("Section .debug_names in %s has abbreviation_table " warning (_("Section .debug_names in %s has abbreviation_table "
"of size %zu vs. written as %u, ignoring .debug_names."), "of size %s vs. written as %u, ignoring .debug_names."),
filename, addr - abbrev_table_start, abbrev_table_size); filename, plongest (addr - abbrev_table_start),
abbrev_table_size);
return false; return false;
} }
map.entry_pool = addr; map.entry_pool = addr;