diff options
| author | Simon Marchi <simon.marchi@polymtl.ca> | 2018-07-21 22:21:57 -0400 |
|---|---|---|
| committer | Simon Marchi <simon.marchi@polymtl.ca> | 2018-07-21 22:23:33 -0400 |
| commit | 17cbafdbbef78d7e05ade53d4cc867fa8ac2f432 (patch) | |
| tree | 38dba452d4deebca5321a41fa7a2f11a2181a1a8 | |
| parent | dcc0660f04bb74948299fb33c98620cfd1ceabb2 (diff) | |
| download | fsf-binutils-gdb-17cbafdbbef78d7e05ade53d4cc867fa8ac2f432.zip fsf-binutils-gdb-17cbafdbbef78d7e05ade53d4cc867fa8ac2f432.tar.gz fsf-binutils-gdb-17cbafdbbef78d7e05ade53d4cc867fa8ac2f432.tar.bz2 | |
Fix -Wunused-variable warnings related to conditional compilation
Some variables are not used when !HAVE_ELF, so avoid declaring them in
that case.
gdb/ChangeLog:
* arm-tdep.c (arm_gdbarch_init): Declare attr_arch and
attr_profile in HAVE_ELF.
* rs6000-tdep.c (bfd_uses_spe_extensions): Declare vector_abi in
HAVE_ELF.
| -rw-r--r-- | gdb/ChangeLog | 7 | ||||
| -rw-r--r-- | gdb/arm-tdep.c | 13 | ||||
| -rw-r--r-- | gdb/rs6000-tdep.c | 5 |
3 files changed, 16 insertions, 9 deletions
diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 68c3bb3..f76f92d 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,10 @@ +2018-07-21 Simon Marchi <simon.marchi@polymtl.ca> + + * arm-tdep.c (arm_gdbarch_init): Declare attr_arch and + attr_profile in HAVE_ELF. + * rs6000-tdep.c (bfd_uses_spe_extensions): Declare vector_abi in + HAVE_ELF. + 2018-07-20 Simon Marchi <simon.marchi@polymtl.ca> * frame.c (frame_register_unwind): Change parameter name. diff --git a/gdb/arm-tdep.c b/gdb/arm-tdep.c index fdfb360..049a887 100644 --- a/gdb/arm-tdep.c +++ b/gdb/arm-tdep.c @@ -8955,7 +8955,6 @@ arm_gdbarch_init (struct gdbarch_info info, struct gdbarch_list *arches) else if (ei_osabi == ELFOSABI_NONE || ei_osabi == ELFOSABI_GNU) { int eabi_ver = EF_ARM_EABI_VERSION (e_flags); - int attr_arch, attr_profile; switch (eabi_ver) { @@ -9026,11 +9025,13 @@ arm_gdbarch_init (struct gdbarch_info info, struct gdbarch_list *arches) executable file includes build attributes; GCC does copy them to the executable, but e.g. RealView does not. */ - attr_arch = bfd_elf_get_obj_attr_int (info.abfd, OBJ_ATTR_PROC, - Tag_CPU_arch); - attr_profile = bfd_elf_get_obj_attr_int (info.abfd, - OBJ_ATTR_PROC, - Tag_CPU_arch_profile); + int attr_arch + = bfd_elf_get_obj_attr_int (info.abfd, OBJ_ATTR_PROC, + Tag_CPU_arch); + int attr_profile + = bfd_elf_get_obj_attr_int (info.abfd, OBJ_ATTR_PROC, + Tag_CPU_arch_profile); + /* GCC specifies the profile for v6-M; RealView only specifies the profile for architectures starting with V7 (as opposed to architectures with a tag diff --git a/gdb/rs6000-tdep.c b/gdb/rs6000-tdep.c index 45bf982..e78de49 100644 --- a/gdb/rs6000-tdep.c +++ b/gdb/rs6000-tdep.c @@ -3558,7 +3558,6 @@ bfd_uses_spe_extensions (bfd *abfd) bfd_size_type size; gdb_byte *ptr; int success = 0; - int vector_abi; if (!abfd) return 0; @@ -3567,8 +3566,8 @@ bfd_uses_spe_extensions (bfd *abfd) /* Using Tag_GNU_Power_ABI_Vector here is a bit of a hack, as the user could be using the SPE vector abi without actually using any spe bits whatsoever. But it's close enough for now. */ - vector_abi = bfd_elf_get_obj_attr_int (abfd, OBJ_ATTR_GNU, - Tag_GNU_Power_ABI_Vector); + int vector_abi = bfd_elf_get_obj_attr_int (abfd, OBJ_ATTR_GNU, + Tag_GNU_Power_ABI_Vector); if (vector_abi == 3) return 1; #endif |