diff mbox

[RFA,02/13] Unused variable fixes related to conditional compilation

Message ID cb4fc10f-950a-d203-db2c-9a5be7d29597@simark.ca
State New, archived
Headers

Commit Message

Simon Marchi July 14, 2018, 9:50 p.m. UTC 
  On 2018-07-13 09:16 PM, Simon Marchi wrote:
> On 2018-07-12 04:51 PM, Tom Tromey wrote:
>> This patch fixes various unused variable warnings that are related to
>> conditional compilation.  In these cases, either the variable is now
>> protected by the same #if as its uses, or the declaration is simply
>> lowered into the conditionally-compiled block.
> 
> This LGTM.
> 
> Simon
> 

I found a few more in a cross-compilation build that has !HAVE_ELF, for
some reason.

From e590cfda6ccb63a5270e36c636fb7daa567aa781 Mon Sep 17 00:00:00 2001
From: Simon Marchi <simon.marchi@polymtl.ca>
Date: Fri, 13 Jul 2018 21:44:52 -0400
Subject: [PATCH] conditional compilation

---
 gdb/arm-tdep.c    | 13 +++++++------
 gdb/rs6000-tdep.c |  5 ++---
 2 files changed, 9 insertions(+), 9 deletions(-)

Comments

Pedro Alves July 16, 2018, 1:45 p.m. UTC | #1 
On 07/14/2018 10:50 PM, Simon Marchi wrote:

> I found a few more in a cross-compilation build that has !HAVE_ELF, for
> some reason.

LGTM.

Thanks,
Pedro Alves
Simon Marchi July 22, 2018, 2:24 a.m. UTC | #2 
On 2018-07-16 09:45 AM, Pedro Alves wrote:
> On 07/14/2018 10:50 PM, Simon Marchi wrote:
> 
>> I found a few more in a cross-compilation build that has !HAVE_ELF, for
>> some reason.
> 
> LGTM.
> 
> Thanks,
> Pedro Alves
> 

I pushed this.

Simon
diff mbox

Patch

diff --git a/gdb/arm-tdep.c b/gdb/arm-tdep.c
index b1a05bbeba2d..c3280ee211d1 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 45bf98267a99..e78de49b2e69 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