diff options
| author | Simon Marchi <simon.marchi@polymtl.ca> | 2021-12-03 16:31:09 -0500 |
|---|---|---|
| committer | Simon Marchi <simon.marchi@polymtl.ca> | 2021-12-14 08:47:48 -0500 |
| commit | 9bc8e54b1f1c281c7a4ecf49a4e9300e334beb0a (patch) | |
| tree | a14d07055ccb22a4858f58147c5492ca55460425 /bfd/elf64-ia64-vms.c | |
| parent | 78d4da9ae0d3447f28274a00b278f58ca7d8d1b2 (diff) | |
| download | gdb-9bc8e54b1f1c281c7a4ecf49a4e9300e334beb0a.zip gdb-9bc8e54b1f1c281c7a4ecf49a4e9300e334beb0a.tar.gz gdb-9bc8e54b1f1c281c7a4ecf49a4e9300e334beb0a.tar.bz2 | |
bfd: fix -Wunused errors with clang 13+
Clang 13 and 14 produce some -Wunused-but-set-{variable,parameter} for
situations where gcc doesn't. In particular, when a variable is set and
then used in a way to update its own value. For example, if `i` is only
used in this way:
int i = 2;
i++;
i = i + 1;
gcc won't warn, but clang will.
Fix all such errors found in an --enable-targets=all build. It would be
important for somebody who knows what they're doing to just make sure
that these variables can indeed be deleted, and that there a no cases
where it's a bug, and the variable should actually be used.
The first instance of this error fix by this patch is:
CC elf32-score.lo
/home/simark/src/binutils-gdb/bfd/elf32-score.c:450:11: error: variable 'relocation' set but not used [-Werror,-Wunused-but-set-variable]
bfd_vma relocation;
^
Change-Id: I2f233ce20352645cf388aff3dfa08a651d21a6b6
Diffstat (limited to 'bfd/elf64-ia64-vms.c')
| -rw-r--r-- | bfd/elf64-ia64-vms.c | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/bfd/elf64-ia64-vms.c b/bfd/elf64-ia64-vms.c index c72015b..54645f0 100644 --- a/bfd/elf64-ia64-vms.c +++ b/bfd/elf64-ia64-vms.c @@ -2512,8 +2512,6 @@ allocate_dynrel_entries (struct elf64_ia64_dyn_sym_info *dyn_i, for (rent = dyn_i->reloc_entries; rent; rent = rent->next) { - int count = rent->count; - switch (rent->type) { case R_IA64_FPTR32LSB: @@ -2538,10 +2536,6 @@ allocate_dynrel_entries (struct elf64_ia64_dyn_sym_info *dyn_i, case R_IA64_IPLTLSB: if (!dynamic_symbol && !shared) continue; - /* Use two REL relocations for IPLT relocations - against local symbols. */ - if (!dynamic_symbol) - count *= 2; break; case R_IA64_DTPREL32LSB: case R_IA64_TPREL64LSB: |