Switch -Wunused-variable on?

Switch -Wunused-variable on?

[Sergio Durigan Junior]

Hello,

While doing something else, I decided to enable the -Wunused-variable
option and... bummer.  So I decided to prepare a patch to fix this,
because there are *many* places with unused variables.

However, before continuing to prepare this patch, I would like what you
think about enabling -Wunused-variable by default in the building.  I am
not sure I can foresee something wrong with it, but maybe others have
some different opinions.

Thanks,

-- 
Sergio

P.S.: I couldn't find any real discussion about this on the list
archives, but please excuse me if this has been already discussed.

Re: Switch -Wunused-variable on?

[Jan Kratochvil]

On Sun, 22 Apr 2012 09:14:44 +0200, Sergio Durigan Junior wrote:
> However, before continuing to prepare this patch, I would like what you
> think about enabling -Wunused-variable by default in the building.

There are more useful -Wunused* options, GDB uses now:
	-Wno-unused -Wunused-value -Wunused-function

IIRC the counterargument for -Wunused-variable was that GDB contains various
#if-compiled code where is a nightmare to get the code right with matching
#if-clauses also for the variables being used by various #if-compiled code
chunks.  I could not find that mail now in the time I gave to it.

I believe more -Wunused* would be worth it, maybe the mail was before Joel
started to disable -Werror for releases so that claim about problematic
#if-clauses is no longer valid.


Thanks,
Jan

Re: Switch -Wunused-variable on?

[Sergio Durigan Junior]

On Sunday, April 22 2012, Jan Kratochvil wrote:

> On Sun, 22 Apr 2012 09:14:44 +0200, Sergio Durigan Junior wrote:
>> However, before continuing to prepare this patch, I would like what you
>> think about enabling -Wunused-variable by default in the building.
>
> There are more useful -Wunused* options, GDB uses now:
> 	-Wno-unused -Wunused-value -Wunused-function

Sorry, do you mean that there are more useful -Wunused* flags than
-Wunused-variable?

> IIRC the counterargument for -Wunused-variable was that GDB contains various
> #if-compiled code where is a nightmare to get the code right with matching
> #if-clauses also for the variables being used by various #if-compiled code
> chunks.  I could not find that mail now in the time I gave to it.

Yes, this is a counter-argument, and I was also thinking about it.  I
don't know how people prefer to deal with this issue, but something
like:

    #define USED_VAR(var) ((void) var)

Would be OK?

> I believe more -Wunused* would be worth it, maybe the mail was before Joel
> started to disable -Werror for releases so that claim about problematic
> #if-clauses is no longer valid.

Yes, I believe the more useful warnings we have, the better.

Thanks,

-- 
Sergio

Re: Switch -Wunused-variable on?

[Jeffrey Walton]

On Sun, Apr 22, 2012 at 4:24 PM, Sergio Durigan Junior
<sergiodj@redhat.com> wrote:
> On Sunday, April 22 2012, Jan Kratochvil wrote:
>
>> On Sun, 22 Apr 2012 09:14:44 +0200, Sergio Durigan Junior wrote:
>>> However, before continuing to prepare this patch, I would like what you
>>> think about enabling -Wunused-variable by default in the building.
>>
>> There are more useful -Wunused* options, GDB uses now:
>>       -Wno-unused -Wunused-value -Wunused-function
>
> Sorry, do you mean that there are more useful -Wunused* flags than
> -Wunused-variable?
>
>> IIRC the counterargument for -Wunused-variable was that GDB contains various
>> #if-compiled code where is a nightmare to get the code right with matching
>> #if-clauses also for the variables being used by various #if-compiled code
>> chunks.  I could not find that mail now in the time I gave to it.
>
> Yes, this is a counter-argument, and I was also thinking about it.  I
> don't know how people prefer to deal with this issue, but something
> like:
>
>    #define USED_VAR(var) ((void) var)
I use this technique when using -Wall. It even works on Visual Studio,
so the technique is portable.

>> I believe more -Wunused* would be worth it, maybe the mail was before Joel
>> started to disable -Werror for releases so that claim about problematic
>> #if-clauses is no longer valid.
>
> Yes, I believe the more useful warnings we have, the better.
In general, you will encounter a lot of resistance to increasing
warning. For example, try to add -Wall -Wextra -Wconversion
-Wstrict-overflow to CFLAGS or CXXFLAGS on some project.

Some even refute the results even when they are valid findings. For
example, when static analysis warned the kernel's sys_prctl was
comparing an unsigned value against less than zero, Jesper Juhl
offered a patch to clean up the code. Linus Torvalds howled “No, you
don't do this... GCC is crap.” For the full discussion, see "[PATCH]
Don't compare unsigned variable for <0 in sys_prctl()"
(http://linux.derkeiler.com/Mailing-Lists/Kernel/2006-11/msg08325.html)
from the Linux Kernel mailing list.

I live and die by GCC's static analysis capabilities. I've got stuff
that has been in the field for years and never taken a bug report or
crashed. But I also validate all parameter in each function (C/C++
makes no guarantees on a call graph), I also check all return values,
assert *everything* in debug builds (the best code is code that debugs
itself), and write negative test cases. I could not care less about
positive test cases since you could hire a kid from a third world
country to copy/paste code that passes positive tests.

If I got pwned as often as the l33t k3rn3l hAck3rz, I would consider
finding a new profession. Worst, the tools are already available to
find an fix most of the low hanging fruit. Arrogance and ignorance is
a big problem in the community...

Jeff

Re: Switch -Wunused-variable on?

[Jan Kratochvil]

On Mon, 23 Apr 2012 00:25:36 +0200, Jeffrey Walton wrote:
> On Sun, Apr 22, 2012 at 4:24 PM, Sergio Durigan Junior <sergiodj@redhat.com> wrote:
> > Yes, this is a counter-argument, and I was also thinking about it.  I
> > don't know how people prefer to deal with this issue,

I would find right to properly #if-compile even the variable declaration but
I do not have real experience how complicated it may be.


> >    #define USED_VAR(var) ((void) var)
> 
> I use this technique when using -Wall. It even works on Visual Studio,
> so the technique is portable.

Sourceware is using ATTRIBUTE_UNUSED.


Thanks,
Jan

Re: Switch -Wunused-variable on?

[Tom Tromey]

>>>>> "Sergio" == Sergio Durigan Junior <sergiodj@redhat.com> writes:

Jan> IIRC the counterargument for -Wunused-variable was that GDB
Jan> contains various #if-compiled code where is a nightmare to get the
Jan> code right with matching #if-clauses also for the variables being
Jan> used by various #if-compiled code chunks.  I could not find that
Jan> mail now in the time I gave to it.

Sergio> Yes, this is a counter-argument, and I was also thinking about it.  I
Sergio> don't know how people prefer to deal with this issue, but something
Sergio> like:
Sergio>     #define USED_VAR(var) ((void) var)
Sergio> Would be OK?

That sort of thing obscures whether the variable later dies.

I'd say that the best approach would be to make the declarations
conditional in the same way that the uses are.

If there are a lot of problem cases with this approach, or if it makes
the code too ugly, then maybe it would be better just not to use this
flag.

Tom

Re: Switch -Wunused-variable on?

[Sergio Durigan Junior]

On Monday, April 23 2012, Tom Tromey wrote:

> I'd say that the best approach would be to make the declarations
> conditional in the same way that the uses are.
>
> If there are a lot of problem cases with this approach, or if it makes
> the code too ugly, then maybe it would be better just not to use this
> flag.

Ok, that makes sense to me.  Since there are tons of places with unused
variables in the code, I will take some time to look and convert the
proper places to use this solution.

Thanks!

-- 
Sergio

Re: Switch -Wunused-variable on?

[Pedro Alves]

On 04/23/2012 04:49 PM, Sergio Durigan Junior wrote:

> On Monday, April 23 2012, Tom Tromey wrote:
> 
>> I'd say that the best approach would be to make the declarations
>> conditional in the same way that the uses are.
>>
>> If there are a lot of problem cases with this approach, or if it makes
>> the code too ugly, then maybe it would be better just not to use this
>> flag.
> 
> Ok, that makes sense to me.  Since there are tons of places with unused
> variables in the code, I will take some time to look and convert the
> proper places to use this solution.


I don't think there'll be that many such cases.

In case it helps, here's how I peek at this sort of thing:

$ .....configure --enable-targets=all
$ make -j8
$ cd gdb
$ make clean
$ make WERROR_CFLAGS="-Wunused-variable" -j8 2>&1 1>/dev/null | tee warnings.txt
...

There aren't _that_ many warnings:

$ grep warning warnings.txt | wc -l
512

(This doesn't catch files only built on other hosts, of course.)

And the worse offenders account for a lot of the instances:

$ grep warning warnings.txt | sed 's/\.c.*/\.c/g' | sort | uniq -c | sort -nr
     50 ../../src/gdb/arm-tdep.c
     15 ../../src/gdb/cris-tdep.c
     13 ../../src/gdb/ia64-tdep.c
     12 ../../src/gdb/dwarf2read.c
     12 ../../src/gdb/c-lang.c
     12 ../../src/gdb/breakpoint.c
     11 ../../src/gdb/mips-tdep.c
     11 ../../src/gdb/m32c-tdep.c
     10 ../../src/gdb/spu-tdep.c
     10 ../../src/gdb/lm32-tdep.c
     10 ../../src/gdb/hppa-tdep.c



I'd welcome this new warning.  I think it's a good idea.



$ cat warnings.txt
Creating observer.itmp
Creating observer.htmp
../../src/gdb/arm-symbian-tdep.c: In function ‘arm_symbian_init_abi’:
../../src/gdb/arm-symbian-tdep.c:66:24: warning: unused variable ‘tdep’ [-Wunused-variable]
../../src/gdb/arm-symbian-tdep.c: In function ‘arm_symbian_osabi_sniffer’:
../../src/gdb/arm-symbian-tdep.c:92:31: warning: unused variable ‘segments’ [-Wunused-variable]
../../src/gdb/bfin-linux-tdep.c: In function ‘bfin_linux_sigframe_init’:
../../src/gdb/bfin-linux-tdep.c:106:19: warning: unused variable ‘this_id’ [-Wunused-variable]
../../src/gdb/bfin-linux-tdep.c:102:19: warning: unused variable ‘gdbarch’ [-Wunused-variable]
../../src/gdb/avr-tdep.c: In function ‘avr_scan_prologue’:
../../src/gdb/avr-tdep.c:745:22: warning: unused variable ‘insn1’ [-Wunused-variable]
../../src/gdb/avr-tdep.c: In function ‘avr_frame_prev_register’:
../../src/gdb/avr-tdep.c:1085:19: warning: unused variable ‘byte_order’ [-Wunused-variable]
../../src/gdb/bfin-tdep.c: In function ‘bfin_push_dummy_call’:
../../src/gdb/bfin-tdep.c:533:20: warning: unused variable ‘arg_type’ [-Wunused-variable]
../../src/gdb/bfin-tdep.c:511:13: warning: unused variable ‘func_addr’ [-Wunused-variable]
../../src/gdb/bfin-tdep.c:510:17: warning: unused variable ‘abi’ [-Wunused-variable]
../../src/gdb/bfin-tdep.c:506:8: warning: unused variable ‘buf’ [-Wunused-variable]
../../src/gdb/bfin-tdep.c:505:24: warning: unused variable ‘tdep’ [-Wunused-variable]
../../src/gdb/cris-tdep.c: In function ‘cris_sigtramp_frame_unwind_cache’:
../../src/gdb/cris-tdep.c:326:13: warning: unused variable ‘sp’ [-Wunused-variable]
../../src/gdb/cris-tdep.c:325:13: warning: unused variable ‘pc’ [-Wunused-variable]
../../src/gdb/cris-tdep.c: In function ‘cris_frame_unwind_cache’:
../../src/gdb/cris-tdep.c:716:7: warning: unused variable ‘i’ [-Wunused-variable]
../../src/gdb/cris-tdep.c:714:13: warning: unused variable ‘pc’ [-Wunused-variable]
../../src/gdb/cris-tdep.c: In function ‘cris_push_dummy_call’:
../../src/gdb/cris-tdep.c:837:13: warning: unused variable ‘fp_mem’ [-Wunused-variable]
../../src/gdb/cris-tdep.c:836:13: warning: unused variable ‘fp_arg’ [-Wunused-variable]
../../src/gdb/cris-tdep.c:831:13: warning: unused variable ‘regval’ [-Wunused-variable]
../../src/gdb/cris-tdep.c:826:7: warning: unused variable ‘stack_alloc’ [-Wunused-variable]
../../src/gdb/cris-tdep.c: In function ‘cris_scan_prologue’:
../../src/gdb/cris-tdep.c:1060:7: warning: unused variable ‘have_fp’ [-Wunused-variable]
../../src/gdb/cris-tdep.c: In function ‘cris_register_size’:
../../src/gdb/cris-tdep.c:1490:24: warning: unused variable ‘tdep’ [-Wunused-variable]
../../src/gdb/cris-tdep.c: In function ‘bdap_prefix’:
../../src/gdb/cris-tdep.c:2193:8: warning: unused variable ‘offset’ [-Wunused-variable]
../../src/gdb/cris-tdep.c: In function ‘cris_supply_gregset’:
../../src/gdb/cris-tdep.c:3851:15: warning: unused variable ‘zerobuf’ [-Wunused-variable]
../../src/gdb/cris-tdep.c: In function ‘_initialize_cris_tdep’:
../../src/gdb/cris-tdep.c:3922:28: warning: unused variable ‘c’ [-Wunused-variable]
../../src/gdb/cris-tdep.c:3920:35: warning: unused variable ‘cris_show_cmdlist’ [-Wunused-variable]
../../src/gdb/cris-tdep.c:3919:35: warning: unused variable ‘cris_set_cmdlist’ [-Wunused-variable]
../../src/gdb/dicos-tdep.c: In function ‘dicos_init_abi’:
../../src/gdb/dicos-tdep.c:31:24: warning: unused variable ‘tdep’ [-Wunused-variable]
In file included from ../../src/gdb/arm-tdep.c:60:0:
../../src/gdb/features/arm-with-m.c: In function ‘initialize_tdesc_arm_with_m’:
../../src/gdb/features/arm-with-m.c:13:35: warning: unused variable ‘type’ [-Wunused-variable]
../../src/gdb/features/arm-with-m.c:13:22: warning: unused variable ‘field_type’ [-Wunused-variable]
In file included from ../../src/gdb/arm-tdep.c:61:0:
../../src/gdb/features/arm-with-m-fpa-layout.c: In function ‘initialize_tdesc_arm_with_m_fpa_layout’:
../../src/gdb/features/arm-with-m-fpa-layout.c:13:35: warning: unused variable ‘type’ [-Wunused-variable]
../../src/gdb/features/arm-with-m-fpa-layout.c:13:22: warning: unused variable ‘field_type’ [-Wunused-variable]
In file included from ../../src/gdb/arm-tdep.c:63:0:
../../src/gdb/features/arm-with-vfpv2.c: In function ‘initialize_tdesc_arm_with_vfpv2’:
../../src/gdb/features/arm-with-vfpv2.c:13:35: warning: unused variable ‘type’ [-Wunused-variable]
../../src/gdb/features/arm-with-vfpv2.c:13:22: warning: unused variable ‘field_type’ [-Wunused-variable]
In file included from ../../src/gdb/arm-tdep.c:64:0:
../../src/gdb/features/arm-with-vfpv3.c: In function ‘initialize_tdesc_arm_with_vfpv3’:
../../src/gdb/features/arm-with-vfpv3.c:13:35: warning: unused variable ‘type’ [-Wunused-variable]
../../src/gdb/features/arm-with-vfpv3.c:13:22: warning: unused variable ‘field_type’ [-Wunused-variable]
In file included from ../../src/gdb/arm-tdep.c:65:0:
../../src/gdb/features/arm-with-neon.c: In function ‘initialize_tdesc_arm_with_neon’:
../../src/gdb/features/arm-with-neon.c:13:35: warning: unused variable ‘type’ [-Wunused-variable]
../../src/gdb/features/arm-with-neon.c:13:22: warning: unused variable ‘field_type’ [-Wunused-variable]
../../src/gdb/arm-tdep.c: In function ‘arm_pc_is_thumb’:
../../src/gdb/arm-tdep.c:382:23: warning: unused variable ‘sec’ [-Wunused-variable]
../../src/gdb/arm-tdep.c: In function ‘arm_skip_stack_protector’:
../../src/gdb/arm-tdep.c:1287:16: warning: unused variable ‘address’ [-Wunused-variable]
../../src/gdb/arm-tdep.c: In function ‘arm_skip_prologue’:
../../src/gdb/arm-tdep.c:1379:26: warning: unused variable ‘sal’ [-Wunused-variable]
../../src/gdb/arm-tdep.c: In function ‘thumb_scan_prologue’:
../../src/gdb/arm-tdep.c:1538:13: warning: unused variable ‘current_pc’ [-Wunused-variable]
../../src/gdb/arm-tdep.c: In function ‘arm_analyze_prologue’:
../../src/gdb/arm-tdep.c:1667:19: warning: unused variable ‘byte_order’ [-Wunused-variable]
../../src/gdb/arm-tdep.c: In function ‘arm_scan_prologue’:
../../src/gdb/arm-tdep.c:1921:13: warning: unused variable ‘offset’ [-Wunused-variable]
../../src/gdb/arm-tdep.c:1920:19: warning: unused variable ‘back_to’ [-Wunused-variable]
../../src/gdb/arm-tdep.c:1919:19: warning: unused variable ‘stack’ [-Wunused-variable]
../../src/gdb/arm-tdep.c:1918:8: warning: unused variable ‘regs’ [-Wunused-variable]
../../src/gdb/arm-tdep.c:1915:43: warning: unused variable ‘current_pc’ [-Wunused-variable]
../../src/gdb/arm-tdep.c:1914:7: warning: unused variable ‘regno’ [-Wunused-variable]
../../src/gdb/arm-tdep.c: In function ‘extend_buffer_earlier’:
../../src/gdb/arm-tdep.c:5134:23: warning: unused variable ‘middle’ [-Wunused-variable]
../../src/gdb/arm-tdep.c: In function ‘arm_adjust_breakpoint_address’:
../../src/gdb/arm-tdep.c:5167:16: warning: unused variable ‘buf2_len’ [-Wunused-variable]
../../src/gdb/arm-tdep.c: In function ‘cleanup_block_load_pc’:
../../src/gdb/arm-tdep.c:6867:67: warning: unused variable ‘i’ [-Wunused-variable]
../../src/gdb/arm-tdep.c: In function ‘arm_copy_block_xfer’:
../../src/gdb/arm-tdep.c:7002:38: warning: unused variable ‘new_rn’ [-Wunused-variable]
../../src/gdb/arm-tdep.c:7002:25: warning: unused variable ‘from’ [-Wunused-variable]
../../src/gdb/arm-tdep.c:7002:17: warning: unused variable ‘to’ [-Wunused-variable]
../../src/gdb/arm-tdep.c:7001:64: warning: unused variable ‘bit’ [-Wunused-variable]
../../src/gdb/arm-tdep.c: In function ‘thumb2_copy_block_xfer’:
../../src/gdb/arm-tdep.c:7104:38: warning: unused variable ‘new_rn’ [-Wunused-variable]
../../src/gdb/arm-tdep.c:7104:25: warning: unused variable ‘from’ [-Wunused-variable]
../../src/gdb/arm-tdep.c:7104:17: warning: unused variable ‘to’ [-Wunused-variable]
../../src/gdb/arm-tdep.c:7103:64: warning: unused variable ‘bit’ [-Wunused-variable]
../../src/gdb/arm-tdep.c: In function ‘arm_decode_miscellaneous’:
../../src/gdb/arm-tdep.c:7409:16: warning: unused variable ‘op1’ [-Wunused-variable]
../../src/gdb/arm-tdep.c: In function ‘arm_decode_ld_st_word_ubyte’:
../../src/gdb/arm-tdep.c:7506:7: warning: unused variable ‘rn_f’ [-Wunused-variable]
../../src/gdb/arm-tdep.c: In function ‘arm_decode_svc_copro’:
../../src/gdb/arm-tdep.c:7701:16: warning: unused variable ‘rn’ [-Wunused-variable]
../../src/gdb/arm-tdep.c: In function ‘thumb2_decode_svc_copro’:
../../src/gdb/arm-tdep.c:7750:16: warning: unused variable ‘rn’ [-Wunused-variable]
../../src/gdb/arm-tdep.c:7746:16: warning: unused variable ‘op1’ [-Wunused-variable]
../../src/gdb/arm-tdep.c: In function ‘thumb_copy_16bit_ldr_literal’:
../../src/gdb/arm-tdep.c:7876:13: warning: unused variable ‘from’ [-Wunused-variable]
../../src/gdb/arm-tdep.c: In function ‘thumb_copy_pop_pc_16bit’:
../../src/gdb/arm-tdep.c:8066:41: warning: unused variable ‘new_rn’ [-Wunused-variable]
../../src/gdb/arm-tdep.c:8066:28: warning: unused variable ‘from’ [-Wunused-variable]
../../src/gdb/arm-tdep.c:8066:20: warning: unused variable ‘to’ [-Wunused-variable]
../../src/gdb/arm-tdep.c:8065:33: warning: unused variable ‘bit’ [-Wunused-variable]
../../src/gdb/arm-tdep.c: In function ‘decode_thumb_32bit_ld_mem_hints’:
../../src/gdb/arm-tdep.c:8203:7: warning: unused variable ‘err’ [-Wunused-variable]
../../src/gdb/arm-tdep.c: In function ‘arm_remote_breakpoint_from_pc’:
../../src/gdb/arm-tdep.c:8709:24: warning: unused variable ‘tdep’ [-Wunused-variable]
../../src/gdb/arm-tdep.c: In function ‘arm_show_fallback_mode’:
../../src/gdb/arm-tdep.c:9284:24: warning: unused variable ‘tdep’ [-Wunused-variable]
../../src/gdb/arm-tdep.c: In function ‘arm_show_force_mode’:
../../src/gdb/arm-tdep.c:9296:24: warning: unused variable ‘tdep’ [-Wunused-variable]
../../src/gdb/arm-tdep.c: In function ‘arm_neon_quad_write’:
../../src/gdb/arm-tdep.c:9575:12: warning: unused variable ‘reg_buf’ [-Wunused-variable]
../../src/gdb/arm-tdep.c: In function ‘_initialize_arm_tdep’:
../../src/gdb/arm-tdep.c:10313:19: warning: unused variable ‘j’ [-Wunused-variable]
../../src/gdb/arm-tdep.c:10309:38: warning: unused variable ‘new_show’ [-Wunused-variable]
../../src/gdb/arm-tdep.c:10309:28: warning: unused variable ‘new_set’ [-Wunused-variable]
../../src/gdb/arm-tdep.c: In function ‘arm_record_extension_space’:
../../src/gdb/arm-tdep.c:10732:56: warning: unused variable ‘tgt_mem_addr’ [-Wunused-variable]
../../src/gdb/arm-tdep.c:10732:42: warning: unused variable ‘offset_8’ [-Wunused-variable]
../../src/gdb/arm-tdep.c:10732:28: warning: unused variable ‘immed_low’ [-Wunused-variable]
../../src/gdb/arm-tdep.c:10732:12: warning: unused variable ‘immed_high’ [-Wunused-variable]
../../src/gdb/arm-tdep.c: In function ‘arm_record_data_proc_misc_ld_str’:
../../src/gdb/arm-tdep.c:11014:57: warning: unused variable ‘tgt_mem_addr’ [-Wunused-variable]
../../src/gdb/arm-tdep.c:11014:43: warning: unused variable ‘offset_8’ [-Wunused-variable]
../../src/gdb/arm-tdep.c:11014:28: warning: unused variable ‘immed_low’ [-Wunused-variable]
../../src/gdb/arm-tdep.c:11014:12: warning: unused variable ‘immed_high’ [-Wunused-variable]
../../src/gdb/arm-tdep.c:11013:26: warning: unused variable ‘reg_src2’ [-Wunused-variable]
../../src/gdb/arm-tdep.c: In function ‘arm_process_record’:
../../src/gdb/arm-tdep.c:12412:19: warning: unused variable ‘byte_order’ [-Wunused-variable]
../../src/gdb/frv-tdep.c: In function ‘new_variant’:
../../src/gdb/frv-tdep.c:139:8: warning: unused variable ‘buf’ [-Wunused-variable]
../../src/gdb/frv-tdep.c: In function ‘frv_frame_unwind_cache’:
../../src/gdb/frv-tdep.c:1103:17: warning: unused variable ‘this_base’ [-Wunused-variable]
../../src/gdb/frv-tdep.c:1102:13: warning: unused variable ‘pc’ [-Wunused-variable]
../../src/gdb/frv-linux-tdep.c: In function ‘frv_linux_sigtramp_frame_cache’:
../../src/gdb/frv-linux-tdep.c:265:24: warning: unused variable ‘tdep’ [-Wunused-variable]
../../src/gdb/h8300-tdep.c: In function ‘h8300_is_argument_spill’:
../../src/gdb/h8300-tdep.c:241:8: warning: unused variable ‘w3’ [-Wunused-variable]
../../src/gdb/h8300-tdep.c: In function ‘h8300_frame_cache’:
../../src/gdb/h8300-tdep.c:433:8: warning: unused variable ‘buf’ [-Wunused-variable]
../../src/gdb/h8300-tdep.c: In function ‘h8300h_extract_return_value’:
../../src/gdb/h8300-tdep.c:786:20: warning: unused variable ‘addr’ [-Wunused-variable]
../../src/gdb/h8300-tdep.c: In function ‘h8300_gdbarch_init’:
../../src/gdb/h8300-tdep.c:1218:24: warning: unused variable ‘tdep’ [-Wunused-variable]
../../src/gdb/h8300-tdep.c: At top level:
../../src/gdb/h8300-tdep.c:941:33: warning: ‘setmachinelist’ defined but not used [-Wunused-variable]
../../src/gdb/hppanbsd-tdep.c: In function ‘hppanbsd_sigtramp_cache_init’:
../../src/gdb/hppanbsd-tdep.c:108:24: warning: unused variable ‘tdep’ [-Wunused-variable]
../../src/gdb/hppanbsd-tdep.c: In function ‘hppanbsd_supply_gregset’:
../../src/gdb/hppanbsd-tdep.c:172:10: warning: unused variable ‘offset’ [-Wunused-variable]
../../src/gdb/hppa-hpux-tdep.c: In function ‘hppa64_hpux_in_solib_call_trampoline’:
../../src/gdb/hppa-hpux-tdep.c:180:13: warning: unused variable ‘i’ [-Wunused-variable]
../../src/gdb/hppaobsd-tdep.c: In function ‘hppaobsd_supply_fpregset’:
../../src/gdb/hppaobsd-tdep.c:77:19: warning: unused variable ‘gdbarch’ [-Wunused-variable]
../../src/gdb/hppa-tdep.c: In function ‘hppa_in_function_epilogue_p’:
../../src/gdb/hppa-tdep.c:546:7: warning: unused variable ‘off’ [-Wunused-variable]
../../src/gdb/hppa-tdep.c: In function ‘hppa32_push_dummy_call’:
../../src/gdb/hppa-tdep.c:703:13: warning: unused variable ‘new_sp’ [-Wunused-variable]
../../src/gdb/hppa-tdep.c: In function ‘after_prologue’:
../../src/gdb/hppa-tdep.c:1701:18: warning: unused variable ‘f’ [-Wunused-variable]
../../src/gdb/hppa-tdep.c: In function ‘hppa_skip_prologue’:
../../src/gdb/hppa-tdep.c:1744:8: warning: unused variable ‘buf’ [-Wunused-variable]
../../src/gdb/hppa-tdep.c:1742:7: warning: unused variable ‘offset’ [-Wunused-variable]
../../src/gdb/hppa-tdep.c:1741:17: warning: unused variable ‘inst’ [-Wunused-variable]
../../src/gdb/hppa-tdep.c: In function ‘hppa_frame_cache’:
../../src/gdb/hppa-tdep.c:1796:13: warning: unused variable ‘this_sp’ [-Wunused-variable]
../../src/gdb/hppa-tdep.c: In function ‘hppa_frame_this_id’:
../../src/gdb/hppa-tdep.c:2195:13: warning: unused variable ‘pc’ [-Wunused-variable]
../../src/gdb/hppa-tdep.c: In function ‘_initialize_hppa_tdep’:
../../src/gdb/hppa-tdep.c:3148:28: warning: unused variable ‘c’ [-Wunused-variable]
../../src/gdb/hppa-tdep.c: At top level:
../../src/gdb/hppa-tdep.c:2785:28: warning: ‘hppa_sigtramp’ defined but not used [-Wunused-variable]
In file included from ../../src/gdb/i386-linux-tdep.c:52:0:
../../src/gdb/features/i386/i386-mmx-linux.c: In function ‘initialize_tdesc_i386_mmx_linux’:
../../src/gdb/features/i386/i386-mmx-linux.c:13:35: warning: unused variable ‘type’ [-Wunused-variable]
../../src/gdb/i386nbsd-tdep.c: In function ‘i386nbsd_sigtramp_cache_init’:
../../src/gdb/i386nbsd-tdep.c:236:24: warning: unused variable ‘tdep’ [-Wunused-variable]
../../src/gdb/i386-nto-tdep.c: In function ‘i386nto_register_area’:
../../src/gdb/i386-nto-tdep.c:137:7: warning: unused variable ‘len’ [-Wunused-variable]
../../src/gdb/i386obsd-tdep.c: In function ‘i386obsd_elf_init_abi’:
../../src/gdb/i386obsd-tdep.c:495:24: warning: unused variable ‘tdep’ [-Wunused-variable]
../../src/gdb/i386-dicos-tdep.c: In function ‘i386_dicos_init_abi’:
../../src/gdb/i386-dicos-tdep.c:46:24: warning: unused variable ‘tdep’ [-Wunused-variable]
../../src/gdb/i387-tdep.c: In function ‘i387_print_float_info’:
../../src/gdb/i387-tdep.c:242:7: warning: unused variable ‘top_p’ [-Wunused-variable]
../../src/gdb/i387-tdep.c:240:7: warning: unused variable ‘fpreg_p’ [-Wunused-variable]
../../src/gdb/i387-tdep.c: In function ‘i387_supply_xsave’:
../../src/gdb/i387-tdep.c:783:19: warning: unused variable ‘p’ [-Wunused-variable]
../../src/gdb/i386-tdep.c: In function ‘i386_match_insn_block’:
../../src/gdb/i386-tdep.c:1193:12: warning: unused variable ‘op’ [-Wunused-variable]
../../src/gdb/linux-tdep.c: In function ‘linux_info_proc’:
../../src/gdb/linux-tdep.c:405:18: warning: unused variable ‘val’ [-Wunused-variable]
../../src/gdb/m32c-tdep.c: In function ‘make_regs’:
../../src/gdb/m32c-tdep.c:866:24: warning: unused variable ‘dra01’ [-Wunused-variable]
../../src/gdb/m32c-tdep.c:865:24: warning: unused variable ‘dsa01’ [-Wunused-variable]
../../src/gdb/m32c-tdep.c:864:24: warning: unused variable ‘dma01’ [-Wunused-variable]
../../src/gdb/m32c-tdep.c:863:24: warning: unused variable ‘drc01’ [-Wunused-variable]
../../src/gdb/m32c-tdep.c:862:24: warning: unused variable ‘dct01’ [-Wunused-variable]
../../src/gdb/m32c-tdep.c:861:24: warning: unused variable ‘dmd01’ [-Wunused-variable]
../../src/gdb/m32c-tdep.c:859:24: warning: unused variable ‘vct’ [-Wunused-variable]
../../src/gdb/m32c-tdep.c:858:24: warning: unused variable ‘svp’ [-Wunused-variable]
../../src/gdb/m32c-tdep.c:857:24: warning: unused variable ‘svf’ [-Wunused-variable]
../../src/gdb/m32c-tdep.c: In function ‘m32c_prev_register’:
../../src/gdb/m32c-tdep.c:1943:7: warning: unused variable ‘reg_size’ [-Wunused-variable]
../../src/gdb/m32c-tdep.c: In function ‘m32c_virtual_frame_pointer’:
../../src/gdb/m32c-tdep.c:2590:34: warning: unused variable ‘sal_end’ [-Wunused-variable]
../../src/gdb/lm32-tdep.c: In function ‘lm32_skip_prologue’:
../../src/gdb/lm32-tdep.c:191:26: warning: unused variable ‘sal’ [-Wunused-variable]
../../src/gdb/lm32-tdep.c: In function ‘lm32_push_dummy_call’:
../../src/gdb/lm32-tdep.c:267:11: warning: unused variable ‘reg’ [-Wunused-variable]
../../src/gdb/lm32-tdep.c:266:11: warning: unused variable ‘j’ [-Wunused-variable]
../../src/gdb/lm32-tdep.c: In function ‘lm32_extract_return_value’:
../../src/gdb/lm32-tdep.c:318:7: warning: unused variable ‘offset’ [-Wunused-variable]
../../src/gdb/lm32-tdep.c: In function ‘lm32_frame_cache’:
../../src/gdb/lm32-tdep.c:435:8: warning: unused variable ‘immediate’ [-Wunused-variable]
../../src/gdb/lm32-tdep.c:433:7: warning: unused variable ‘offsets’ [-Wunused-variable]
../../src/gdb/lm32-tdep.c:432:7: warning: unused variable ‘op’ [-Wunused-variable]
../../src/gdb/lm32-tdep.c:431:17: warning: unused variable ‘instruction’ [-Wunused-variable]
../../src/gdb/lm32-tdep.c:430:7: warning: unused variable ‘prefixed’ [-Wunused-variable]
../../src/gdb/lm32-tdep.c:425:13: warning: unused variable ‘prologue_pc’ [-Wunused-variable]
../../src/gdb/m32r-linux-tdep.c: In function ‘m32r_linux_regset_from_core_section’:
../../src/gdb/m32r-linux-tdep.c:399:24: warning: unused variable ‘tdep’ [-Wunused-variable]
../../src/gdb/m32r-linux-tdep.c: In function ‘m32r_linux_init_abi’:
../../src/gdb/m32r-linux-tdep.c:408:24: warning: unused variable ‘tdep’ [-Wunused-variable]
../../src/gdb/m32r-tdep.c: In function ‘m32r_frame_unwind_cache’:
../../src/gdb/m32r-tdep.c:540:21: warning: unused variable ‘op2’ [-Wunused-variable]
../../src/gdb/m32r-tdep.c: In function ‘m32r_push_dummy_call’:
../../src/gdb/m32r-tdep.c:700:7: warning: unused variable ‘odd_sized_struct’ [-Wunused-variable]
../../src/gdb/m68klinux-tdep.c: In function ‘m68k_linux_pc_in_sigtramp’:
../../src/gdb/m68klinux-tdep.c:71:13: warning: unused variable ‘sp’ [-Wunused-variable]
../../src/gdb/m68klinux-tdep.c: In function ‘m68k_linux_sigtramp_frame_cache’:
../../src/gdb/m68klinux-tdep.c:267:24: warning: unused variable ‘tdep’ [-Wunused-variable]
../../src/gdb/m68k-tdep.c: In function ‘m68k_skip_prologue’:
../../src/gdb/m68k-tdep.c:857:7: warning: unused variable ‘op’ [-Wunused-variable]
../../src/gdb/microblaze-tdep.c: In function ‘microblaze_alloc_frame_cache’:
../../src/gdb/microblaze-tdep.c:176:7: warning: unused variable ‘i’ [-Wunused-variable]
../../src/gdb/microblaze-tdep.c: In function ‘microblaze_analyze_prologue’:
../../src/gdb/microblaze-tdep.c:234:7: warning: unused variable ‘rn’ [-Wunused-variable]
../../src/gdb/microblaze-tdep.c: In function ‘microblaze_frame_cache’:
../../src/gdb/microblaze-tdep.c:464:23: warning: unused variable ‘fp’ [-Wunused-variable]
../../src/gdb/microblaze-tdep.c:464:19: warning: unused variable ‘pc’ [-Wunused-variable]
../../src/gdb/microblaze-linux-tdep.c: In function ‘microblaze_linux_sigtramp_cache’:
../../src/gdb/microblaze-linux-tdep.c:76:24: warning: unused variable ‘tdep’ [-Wunused-variable]
../../src/gdb/microblaze-linux-tdep.c: In function ‘microblaze_linux_init_abi’:
../../src/gdb/microblaze-linux-tdep.c:124:24: warning: unused variable ‘tdep’ [-Wunused-variable]
../../src/gdb/mep-tdep.c: In function ‘mep_register_name’:
../../src/gdb/mep-tdep.c:932:24: warning: unused variable ‘tdep’ [-Wunused-variable]
../../src/gdb/mep-tdep.c: In function ‘mep_analyze_prologue’:
../../src/gdb/mep-tdep.c:1689:7: warning: unused variable ‘found_lp’ [-Wunused-variable]
../../src/gdb/mep-tdep.c: In function ‘mep_push_dummy_call’:
../../src/gdb/mep-tdep.c:2307:13: warning: unused variable ‘func_addr’ [-Wunused-variable]
../../src/gdb/mips-linux-tdep.c: In function ‘mips_fill_fpregset’:
../../src/gdb/mips-linux-tdep.c:249:9: warning: unused variable ‘from’ [-Wunused-variable]
../../src/gdb/mips-linux-tdep.c: In function ‘mips_linux_o32_sigframe_init’:
../../src/gdb/mips-linux-tdep.c:939:13: warning: unused variable ‘reg_position’ [-Wunused-variable]
../../src/gdb/mips-linux-tdep.c: In function ‘mips_linux_n32n64_sigframe_init’:
../../src/gdb/mips-linux-tdep.c:1105:13: warning: unused variable ‘reg_position’ [-Wunused-variable]
../../src/gdb/mips-tdep.c: In function ‘mips_pseudo_register_type’:
../../src/gdb/mips-tdep.c:913:24: warning: unused variable ‘tdep’ [-Wunused-variable]
../../src/gdb/mips-tdep.c: In function ‘extended_mips16_next_pc’:
../../src/gdb/mips-tdep.c:1518:12: warning: unused variable ‘offset’ [-Wunused-variable]
../../src/gdb/mips-tdep.c: In function ‘mips_eabi_push_dummy_call’:
../../src/gdb/mips-tdep.c:2965:24: warning: unused variable ‘tdep’ [-Wunused-variable]
../../src/gdb/mips-tdep.c: In function ‘mips_n32n64_push_dummy_call’:
../../src/gdb/mips-tdep.c:3373:24: warning: unused variable ‘tdep’ [-Wunused-variable]
../../src/gdb/mips-tdep.c: In function ‘mips_o32_push_dummy_call’:
../../src/gdb/mips-tdep.c:3816:24: warning: unused variable ‘tdep’ [-Wunused-variable]
../../src/gdb/mips-tdep.c: In function ‘mips_o64_push_dummy_call’:
../../src/gdb/mips-tdep.c:4284:24: warning: unused variable ‘tdep’ [-Wunused-variable]
../../src/gdb/mips-tdep.c: In function ‘mips_o64_return_value’:
../../src/gdb/mips-tdep.c:4528:24: warning: unused variable ‘tdep’ [-Wunused-variable]
../../src/gdb/mips-tdep.c: In function ‘mips_print_register’:
../../src/gdb/mips-tdep.c:4767:7: warning: unused variable ‘offset’ [-Wunused-variable]
../../src/gdb/mips-tdep.c: In function ‘mips_adjust_breakpoint_address’:
../../src/gdb/mips-tdep.c:5558:30: warning: unused variable ‘sym’ [-Wunused-variable]
../../src/gdb/mips-tdep.c:5504:24: warning: unused variable ‘next_addr’ [-Wunused-variable]
../../src/gdb/mips-tdep.c: In function ‘_initialize_mips_tdep’:
../../src/gdb/mips-tdep.c:6836:28: warning: unused variable ‘c’ [-Wunused-variable]
../../src/gdb/mn10300-tdep.c: In function ‘mn10300_analyze_prologue’:
../../src/gdb/mn10300-tdep.c:390:17: warning: unused variable ‘next_pc’ [-Wunused-variable]
../../src/gdb/mn10300-tdep.c: In function ‘mn10300_frame_prev_register’:
../../src/gdb/mn10300-tdep.c:1158:7: warning: unused variable ‘reg_size’ [-Wunused-variable]
../../src/gdb/mn10300-tdep.c:1154:24: warning: unused variable ‘tdep’ [-Wunused-variable]
../../src/gdb/ppc-sysv-tdep.c: In function ‘convert_code_addr_to_desc_addr’:
../../src/gdb/ppc-sysv-tdep.c:1080:13: warning: unused variable ‘toc’ [-Wunused-variable]
../../src/gdb/rl78-tdep.c: In function ‘rl78_push_dummy_call’:
../../src/gdb/rl78-tdep.c:1024:11: warning: unused variable ‘offset’ [-Wunused-variable]
In file included from ../../src/gdb/ppc-linux-tdep.c:53:0:
../../src/gdb/features/rs6000/powerpc-32l.c: In function ‘initialize_tdesc_powerpc_32l’:
../../src/gdb/features/rs6000/powerpc-32l.c:12:35: warning: unused variable ‘type’ [-Wunused-variable]
../../src/gdb/features/rs6000/powerpc-32l.c:12:22: warning: unused variable ‘field_type’ [-Wunused-variable]
In file included from ../../src/gdb/ppc-linux-tdep.c:57:0:
../../src/gdb/features/rs6000/powerpc-isa205-32l.c: In function ‘initialize_tdesc_powerpc_isa205_32l’:
../../src/gdb/features/rs6000/powerpc-isa205-32l.c:12:35: warning: unused variable ‘type’ [-Wunused-variable]
../../src/gdb/features/rs6000/powerpc-isa205-32l.c:12:22: warning: unused variable ‘field_type’ [-Wunused-variable]
In file included from ../../src/gdb/ppc-linux-tdep.c:60:0:
../../src/gdb/features/rs6000/powerpc-64l.c: In function ‘initialize_tdesc_powerpc_64l’:
../../src/gdb/features/rs6000/powerpc-64l.c:12:35: warning: unused variable ‘type’ [-Wunused-variable]
../../src/gdb/features/rs6000/powerpc-64l.c:12:22: warning: unused variable ‘field_type’ [-Wunused-variable]
In file included from ../../src/gdb/ppc-linux-tdep.c:64:0:
../../src/gdb/features/rs6000/powerpc-isa205-64l.c: In function ‘initialize_tdesc_powerpc_isa205_64l’:
../../src/gdb/features/rs6000/powerpc-isa205-64l.c:12:35: warning: unused variable ‘type’ [-Wunused-variable]
../../src/gdb/features/rs6000/powerpc-isa205-64l.c:12:22: warning: unused variable ‘field_type’ [-Wunused-variable]
In file included from ../../src/gdb/ppc-linux-tdep.c:67:0:
../../src/gdb/features/rs6000/powerpc-e500l.c: In function ‘initialize_tdesc_powerpc_e500l’:
../../src/gdb/features/rs6000/powerpc-e500l.c:12:35: warning: unused variable ‘type’ [-Wunused-variable]
../../src/gdb/features/rs6000/powerpc-e500l.c:12:22: warning: unused variable ‘field_type’ [-Wunused-variable]
../../src/gdb/ppc-linux-tdep.c: In function ‘powerpc_linux_in_dynsym_resolve_code’:
../../src/gdb/ppc-linux-tdep.c:634:19: warning: unused variable ‘objfile’ [-Wunused-variable]
../../src/gdb/rs6000-aix-tdep.c: In function ‘rs6000_push_dummy_call’:
../../src/gdb/rs6000-aix-tdep.c:299:8: warning: unused variable ‘adj’ [-Wunused-variable]
../../src/gdb/rs6000-aix-tdep.c: In function ‘rs6000_return_value’:
../../src/gdb/rs6000-aix-tdep.c:433:12: warning: unused variable ‘buf’ [-Wunused-variable]
In file included from ../../src/gdb/rs6000-tdep.c:63:0:
../../src/gdb/features/rs6000/powerpc-32.c: In function ‘initialize_tdesc_powerpc_32’:
../../src/gdb/features/rs6000/powerpc-32.c:12:35: warning: unused variable ‘type’ [-Wunused-variable]
../../src/gdb/features/rs6000/powerpc-32.c:12:22: warning: unused variable ‘field_type’ [-Wunused-variable]
In file included from ../../src/gdb/rs6000-tdep.c:66:0:
../../src/gdb/features/rs6000/powerpc-403.c: In function ‘initialize_tdesc_powerpc_403’:
../../src/gdb/features/rs6000/powerpc-403.c:12:35: warning: unused variable ‘type’ [-Wunused-variable]
../../src/gdb/features/rs6000/powerpc-403.c:12:22: warning: unused variable ‘field_type’ [-Wunused-variable]
In file included from ../../src/gdb/rs6000-tdep.c:67:0:
../../src/gdb/features/rs6000/powerpc-403gc.c: In function ‘initialize_tdesc_powerpc_403gc’:
../../src/gdb/features/rs6000/powerpc-403gc.c:12:35: warning: unused variable ‘type’ [-Wunused-variable]
../../src/gdb/features/rs6000/powerpc-403gc.c:12:22: warning: unused variable ‘field_type’ [-Wunused-variable]
In file included from ../../src/gdb/rs6000-tdep.c:68:0:
../../src/gdb/features/rs6000/powerpc-405.c: In function ‘initialize_tdesc_powerpc_405’:
../../src/gdb/features/rs6000/powerpc-405.c:13:29: warning: unused variable ‘type’ [-Wunused-variable]
../../src/gdb/features/rs6000/powerpc-405.c:13:16: warning: unused variable ‘field_type’ [-Wunused-variable]
In file included from ../../src/gdb/rs6000-tdep.c:69:0:
../../src/gdb/features/rs6000/powerpc-505.c: In function ‘initialize_tdesc_powerpc_505’:
../../src/gdb/features/rs6000/powerpc-505.c:12:35: warning: unused variable ‘type’ [-Wunused-variable]
../../src/gdb/features/rs6000/powerpc-505.c:12:22: warning: unused variable ‘field_type’ [-Wunused-variable]
In file included from ../../src/gdb/rs6000-tdep.c:70:0:
../../src/gdb/features/rs6000/powerpc-601.c: In function ‘initialize_tdesc_powerpc_601’:
../../src/gdb/features/rs6000/powerpc-601.c:12:35: warning: unused variable ‘type’ [-Wunused-variable]
../../src/gdb/features/rs6000/powerpc-601.c:12:22: warning: unused variable ‘field_type’ [-Wunused-variable]
In file included from ../../src/gdb/rs6000-tdep.c:71:0:
../../src/gdb/features/rs6000/powerpc-602.c: In function ‘initialize_tdesc_powerpc_602’:
../../src/gdb/features/rs6000/powerpc-602.c:12:35: warning: unused variable ‘type’ [-Wunused-variable]
../../src/gdb/features/rs6000/powerpc-602.c:12:22: warning: unused variable ‘field_type’ [-Wunused-variable]
In file included from ../../src/gdb/rs6000-tdep.c:72:0:
../../src/gdb/features/rs6000/powerpc-603.c: In function ‘initialize_tdesc_powerpc_603’:
../../src/gdb/features/rs6000/powerpc-603.c:12:35: warning: unused variable ‘type’ [-Wunused-variable]
../../src/gdb/features/rs6000/powerpc-603.c:12:22: warning: unused variable ‘field_type’ [-Wunused-variable]
In file included from ../../src/gdb/rs6000-tdep.c:73:0:
../../src/gdb/features/rs6000/powerpc-604.c: In function ‘initialize_tdesc_powerpc_604’:
../../src/gdb/features/rs6000/powerpc-604.c:12:35: warning: unused variable ‘type’ [-Wunused-variable]
../../src/gdb/features/rs6000/powerpc-604.c:12:22: warning: unused variable ‘field_type’ [-Wunused-variable]
In file included from ../../src/gdb/rs6000-tdep.c:74:0:
../../src/gdb/features/rs6000/powerpc-64.c: In function ‘initialize_tdesc_powerpc_64’:
../../src/gdb/features/rs6000/powerpc-64.c:12:35: warning: unused variable ‘type’ [-Wunused-variable]
../../src/gdb/features/rs6000/powerpc-64.c:12:22: warning: unused variable ‘field_type’ [-Wunused-variable]
In file included from ../../src/gdb/rs6000-tdep.c:78:0:
../../src/gdb/features/rs6000/powerpc-750.c: In function ‘initialize_tdesc_powerpc_750’:
../../src/gdb/features/rs6000/powerpc-750.c:12:35: warning: unused variable ‘type’ [-Wunused-variable]
../../src/gdb/features/rs6000/powerpc-750.c:12:22: warning: unused variable ‘field_type’ [-Wunused-variable]
In file included from ../../src/gdb/rs6000-tdep.c:79:0:
../../src/gdb/features/rs6000/powerpc-860.c: In function ‘initialize_tdesc_powerpc_860’:
../../src/gdb/features/rs6000/powerpc-860.c:12:35: warning: unused variable ‘type’ [-Wunused-variable]
../../src/gdb/features/rs6000/powerpc-860.c:12:22: warning: unused variable ‘field_type’ [-Wunused-variable]
In file included from ../../src/gdb/rs6000-tdep.c:80:0:
../../src/gdb/features/rs6000/powerpc-e500.c: In function ‘initialize_tdesc_powerpc_e500’:
../../src/gdb/features/rs6000/powerpc-e500.c:12:35: warning: unused variable ‘type’ [-Wunused-variable]
../../src/gdb/features/rs6000/powerpc-e500.c:12:22: warning: unused variable ‘field_type’ [-Wunused-variable]
In file included from ../../src/gdb/rs6000-tdep.c:81:0:
../../src/gdb/features/rs6000/rs6000.c: In function ‘initialize_tdesc_rs6000’:
../../src/gdb/features/rs6000/rs6000.c:12:35: warning: unused variable ‘type’ [-Wunused-variable]
../../src/gdb/features/rs6000/rs6000.c:12:22: warning: unused variable ‘field_type’ [-Wunused-variable]
../../src/gdb/rs6000-tdep.c: In function ‘ppc_deal_with_atomic_sequence’:
../../src/gdb/rs6000-tdep.c:1098:7: warning: unused variable ‘opcode’ [-Wunused-variable]
../../src/gdb/rs6000-tdep.c: In function ‘rs6000_gdbarch_init’:
../../src/gdb/rs6000-tdep.c:3655:32: warning: unused variable ‘segment_regs’ [-Wunused-variable]
../../src/gdb/rs6000-tdep.c:3555:13: warning: unused variable ‘sect’ [-Wunused-variable]
../../src/gdb/rx-tdep.c: In function ‘rx_frame_prev_register’:
../../src/gdb/rx-tdep.c:439:7: warning: unused variable ‘reg_size’ [-Wunused-variable]
../../src/gdb/sh64-tdep.c: In function ‘sh64_analyze_prologue’:
../../src/gdb/sh64-tdep.c:869:24: warning: unused variable ‘tdep’ [-Wunused-variable]
../../src/gdb/sh64-tdep.c:868:7: warning: unused variable ‘register_number’ [-Wunused-variable]
../../src/gdb/sh64-tdep.c:867:7: warning: unused variable ‘gdb_register_number’ [-Wunused-variable]
../../src/gdb/sh64-tdep.c:861:7: warning: unused variable ‘reg_nr’ [-Wunused-variable]
../../src/gdb/sh64-tdep.c: In function ‘sh64_push_dummy_call’:
../../src/gdb/sh64-tdep.c:1073:8: warning: unused variable ‘valbuf_tmp’ [-Wunused-variable]
../../src/gdb/sh64-tdep.c: In function ‘sh64_extract_return_value’:
../../src/gdb/sh64-tdep.c:1232:19: warning: unused variable ‘byte_order’ [-Wunused-variable]
../../src/gdb/score-tdep.c: In function ‘score3_adjust_pc_and_fetch_inst’:
../../src/gdb/score-tdep.c:224:7: warning: unused variable ‘inst_len’ [-Wunused-variable]
../../src/gdb/score-tdep.c: In function ‘score3_analyze_prologue’:
../../src/gdb/score-tdep.c:1182:18: warning: unused variable ‘save_v’ [-Wunused-variable]
../../src/gdb/score-tdep.c:1268:18: warning: unused variable ‘save_v’ [-Wunused-variable]
In file included from ../../src/gdb/s390-tdep.c:48:0:
../../src/gdb/features/s390-linux32.c: In function ‘initialize_tdesc_s390_linux32’:
../../src/gdb/features/s390-linux32.c:13:35: warning: unused variable ‘type’ [-Wunused-variable]
../../src/gdb/features/s390-linux32.c:13:22: warning: unused variable ‘field_type’ [-Wunused-variable]
In file included from ../../src/gdb/s390-tdep.c:49:0:
../../src/gdb/features/s390-linux32v1.c: In function ‘initialize_tdesc_s390_linux32v1’:
../../src/gdb/features/s390-linux32v1.c:13:35: warning: unused variable ‘type’ [-Wunused-variable]
../../src/gdb/features/s390-linux32v1.c:13:22: warning: unused variable ‘field_type’ [-Wunused-variable]
In file included from ../../src/gdb/s390-tdep.c:50:0:
../../src/gdb/features/s390-linux32v2.c: In function ‘initialize_tdesc_s390_linux32v2’:
../../src/gdb/features/s390-linux32v2.c:13:35: warning: unused variable ‘type’ [-Wunused-variable]
../../src/gdb/features/s390-linux32v2.c:13:22: warning: unused variable ‘field_type’ [-Wunused-variable]
In file included from ../../src/gdb/s390-tdep.c:51:0:
../../src/gdb/features/s390-linux64.c: In function ‘initialize_tdesc_s390_linux64’:
../../src/gdb/features/s390-linux64.c:13:35: warning: unused variable ‘type’ [-Wunused-variable]
../../src/gdb/features/s390-linux64.c:13:22: warning: unused variable ‘field_type’ [-Wunused-variable]
In file included from ../../src/gdb/s390-tdep.c:52:0:
../../src/gdb/features/s390-linux64v1.c: In function ‘initialize_tdesc_s390_linux64v1’:
../../src/gdb/features/s390-linux64v1.c:13:35: warning: unused variable ‘type’ [-Wunused-variable]
../../src/gdb/features/s390-linux64v1.c:13:22: warning: unused variable ‘field_type’ [-Wunused-variable]
In file included from ../../src/gdb/s390-tdep.c:53:0:
../../src/gdb/features/s390-linux64v2.c: In function ‘initialize_tdesc_s390_linux64v2’:
../../src/gdb/features/s390-linux64v2.c:13:35: warning: unused variable ‘type’ [-Wunused-variable]
../../src/gdb/features/s390-linux64v2.c:13:22: warning: unused variable ‘field_type’ [-Wunused-variable]
In file included from ../../src/gdb/s390-tdep.c:54:0:
../../src/gdb/features/s390x-linux64.c: In function ‘initialize_tdesc_s390x_linux64’:
../../src/gdb/features/s390x-linux64.c:13:35: warning: unused variable ‘type’ [-Wunused-variable]
../../src/gdb/features/s390x-linux64.c:13:22: warning: unused variable ‘field_type’ [-Wunused-variable]
In file included from ../../src/gdb/s390-tdep.c:55:0:
../../src/gdb/features/s390x-linux64v1.c: In function ‘initialize_tdesc_s390x_linux64v1’:
../../src/gdb/features/s390x-linux64v1.c:13:35: warning: unused variable ‘type’ [-Wunused-variable]
../../src/gdb/features/s390x-linux64v1.c:13:22: warning: unused variable ‘field_type’ [-Wunused-variable]
In file included from ../../src/gdb/s390-tdep.c:56:0:
../../src/gdb/features/s390x-linux64v2.c: In function ‘initialize_tdesc_s390x_linux64v2’:
../../src/gdb/features/s390x-linux64v2.c:13:35: warning: unused variable ‘type’ [-Wunused-variable]
../../src/gdb/features/s390x-linux64v2.c:13:22: warning: unused variable ‘field_type’ [-Wunused-variable]
../../src/gdb/s390-tdep.c: In function ‘s390_load’:
../../src/gdb/s390-tdep.c:1163:8: warning: unused variable ‘offset’ [-Wunused-variable]
../../src/gdb/s390-tdep.c: In function ‘s390_frame_prev_register’:
../../src/gdb/s390-tdep.c:2027:19: warning: unused variable ‘gdbarch’ [-Wunused-variable]
../../src/gdb/s390-tdep.c: In function ‘s390_dwarf2_frame_init_reg’:
../../src/gdb/s390-tdep.c:2349:24: warning: unused variable ‘tdep’ [-Wunused-variable]
../../src/gdb/sh-tdep.c: In function ‘sh_treat_as_flt_p’:
../../src/gdb/sh-tdep.c:1037:7: warning: unused variable ‘len’ [-Wunused-variable]
../../src/gdb/sh-tdep.c: In function ‘sh_extract_return_value_nofpu’:
../../src/gdb/sh-tdep.c:1313:7: warning: unused variable ‘offset’ [-Wunused-variable]
../../src/gdb/sh-tdep.c:1312:7: warning: unused variable ‘return_register’ [-Wunused-variable]
../../src/gdb/sh-tdep.c: In function ‘sh2a_nofpu_show_regs’:
../../src/gdb/sh-tdep.c:1718:7: warning: unused variable ‘pr’ [-Wunused-variable]
../../src/gdb/sh-tdep.c: In function ‘_initialize_sh_tdep’:
../../src/gdb/sh-tdep.c:3161:28: warning: unused variable ‘c’ [-Wunused-variable]
../../src/gdb/sparcobsd-tdep.c: In function ‘sparc32obsd_init_abi’:
../../src/gdb/sparcobsd-tdep.c:239:24: warning: unused variable ‘tdep’ [-Wunused-variable]
../../src/gdb/spu-tdep.c: In function ‘spu_in_function_epilogue_p’:
../../src/gdb/spu-tdep.c:852:19: warning: unused variable ‘rc’ [-Wunused-variable]
../../src/gdb/spu-tdep.c: In function ‘spu_get_overlay_table’:
../../src/gdb/spu-tdep.c:1784:17: warning: unused variable ‘size’ [-Wunused-variable]
../../src/gdb/spu-tdep.c: In function ‘spu_overlay_new_objfile’:
../../src/gdb/spu-tdep.c:1880:12: warning: unused variable ‘obfd’ [-Wunused-variable]
../../src/gdb/spu-tdep.c: In function ‘info_spu_event_command’:
../../src/gdb/spu-tdep.c:2033:7: warning: unused variable ‘rc’ [-Wunused-variable]
../../src/gdb/spu-tdep.c: In function ‘info_spu_signal_command’:
../../src/gdb/spu-tdep.c:2090:7: warning: unused variable ‘rc’ [-Wunused-variable]
../../src/gdb/spu-tdep.c: In function ‘info_spu_mailbox_command’:
../../src/gdb/spu-tdep.c:2211:7: warning: unused variable ‘i’ [-Wunused-variable]
../../src/gdb/spu-tdep.c: In function ‘info_spu_dma_cmdlist’:
../../src/gdb/spu-tdep.c:2359:16: warning: unused variable ‘size’ [-Wunused-variable]
../../src/gdb/spu-tdep.c:2359:11: warning: unused variable ‘lsa’ [-Wunused-variable]
../../src/gdb/spu-tdep.c: In function ‘info_spu_dma_command’:
../../src/gdb/spu-tdep.c:2453:7: warning: unused variable ‘i’ [-Wunused-variable]
../../src/gdb/spu-tdep.c: In function ‘info_spu_proxydma_command’:
../../src/gdb/spu-tdep.c:2532:7: warning: unused variable ‘i’ [-Wunused-variable]
In file included from ../../src/gdb/tic6x-tdep.c:53:0:
../../src/gdb/features/tic6x-c64xp.c: In function ‘initialize_tdesc_tic6x_c64xp’:
../../src/gdb/features/tic6x-c64xp.c:13:35: warning: unused variable ‘type’ [-Wunused-variable]
../../src/gdb/features/tic6x-c64xp.c:13:22: warning: unused variable ‘field_type’ [-Wunused-variable]
In file included from ../../src/gdb/tic6x-tdep.c:54:0:
../../src/gdb/features/tic6x-c64x.c: In function ‘initialize_tdesc_tic6x_c64x’:
../../src/gdb/features/tic6x-c64x.c:13:35: warning: unused variable ‘type’ [-Wunused-variable]
../../src/gdb/features/tic6x-c64x.c:13:22: warning: unused variable ‘field_type’ [-Wunused-variable]
In file included from ../../src/gdb/tic6x-tdep.c:55:0:
../../src/gdb/features/tic6x-c62x.c: In function ‘initialize_tdesc_tic6x_c62x’:
../../src/gdb/features/tic6x-c62x.c:13:35: warning: unused variable ‘type’ [-Wunused-variable]
../../src/gdb/features/tic6x-c62x.c:13:22: warning: unused variable ‘field_type’ [-Wunused-variable]
../../src/gdb/tic6x-tdep.c: In function ‘tic6x_analyze_prologue’:
../../src/gdb/tic6x-tdep.c:179:11: warning: unused variable ‘offset’ [-Wunused-variable]
../../src/gdb/tic6x-tdep.c:155:19: warning: unused variable ‘byte_order’ [-Wunused-variable]
../../src/gdb/tic6x-tdep.c: In function ‘tic6x_skip_prologue’:
../../src/gdb/tic6x-tdep.c:304:13: warning: unused variable ‘limit_pc’ [-Wunused-variable]
../../src/gdb/tic6x-tdep.c: In function ‘tic6x_frame_unwind_cache’:
../../src/gdb/tic6x-tdep.c:408:7: warning: unused variable ‘i’ [-Wunused-variable]
../../src/gdb/tic6x-tdep.c: In function ‘tic6x_get_next_pc’:
../../src/gdb/tic6x-tdep.c:609:7: warning: unused variable ‘offset’ [-Wunused-variable]
../../src/gdb/tic6x-tdep.c: In function ‘tic6x_push_dummy_call’:
../../src/gdb/tic6x-tdep.c:912:13: warning: unused variable ‘func_addr’ [-Wunused-variable]
../../src/gdb/tic6x-tdep.c:909:7: warning: unused variable ‘len’ [-Wunused-variable]
../../src/gdb/sparc-tdep.c: In function ‘sparc_alloc_frame_cache’:
../../src/gdb/sparc-tdep.c:605:7: warning: unused variable ‘i’ [-Wunused-variable]
../../src/gdb/sparc-tdep.c: In function ‘sparc_skip_stack_check’:
../../src/gdb/sparc-tdep.c:633:7: warning: unused variable ‘offset_stack_checking_sequence’ [-Wunused-variable]
../../src/gdb/spu-multiarch.c: In function ‘parse_spufs_run’:
../../src/gdb/spu-multiarch.c:64:13: warning: unused variable ‘pc’ [-Wunused-variable]
In file included from ../../src/gdb/tic6x-linux-tdep.c:31:0:
../../src/gdb/features/tic6x-c64xp-linux.c: In function ‘initialize_tdesc_tic6x_c64xp_linux’:
../../src/gdb/features/tic6x-c64xp-linux.c:13:35: warning: unused variable ‘type’ [-Wunused-variable]
../../src/gdb/features/tic6x-c64xp-linux.c:13:22: warning: unused variable ‘field_type’ [-Wunused-variable]
In file included from ../../src/gdb/tic6x-linux-tdep.c:32:0:
../../src/gdb/features/tic6x-c64x-linux.c: In function ‘initialize_tdesc_tic6x_c64x_linux’:
../../src/gdb/features/tic6x-c64x-linux.c:13:35: warning: unused variable ‘type’ [-Wunused-variable]
../../src/gdb/features/tic6x-c64x-linux.c:13:22: warning: unused variable ‘field_type’ [-Wunused-variable]
In file included from ../../src/gdb/tic6x-linux-tdep.c:33:0:
../../src/gdb/features/tic6x-c62x-linux.c: In function ‘initialize_tdesc_tic6x_c62x_linux’:
../../src/gdb/features/tic6x-c62x-linux.c:13:35: warning: unused variable ‘type’ [-Wunused-variable]
../../src/gdb/features/tic6x-c62x-linux.c:13:22: warning: unused variable ‘field_type’ [-Wunused-variable]
../../src/gdb/v850-tdep.c: In function ‘v850_analyze_prologue’:
../../src/gdb/v850-tdep.c:596:7: warning: unused variable ‘fp_used’ [-Wunused-variable]
../../src/gdb/v850-tdep.c: In function ‘v850_alloc_frame_cache’:
../../src/gdb/v850-tdep.c:961:7: warning: unused variable ‘i’ [-Wunused-variable]
../../src/gdb/xtensa-tdep.c: In function ‘xtensa_pseudo_register_write’:
../../src/gdb/xtensa-tdep.c:668:20: warning: unused variable ‘wb’ [-Wunused-variable]
../../src/gdb/xtensa-tdep.c: In function ‘execute_code’:
../../src/gdb/xtensa-tdep.c:2819:7: warning: unused variable ‘num_operands’ [-Wunused-variable]
../../src/gdb/xtensa-tdep.c:2815:7: warning: unused variable ‘fail’ [-Wunused-variable]
../../src/gdb/xtensa-tdep.c: In function ‘xtensa_gdbarch_init’:
../../src/gdb/xtensa-tdep.c:3212:30: warning: unused variable ‘abi_handler’ [-Wunused-variable]
../../src/gdb/xtensa-tdep.c: In function ‘_initialize_xtensa_tdep’:
../../src/gdb/xtensa-tdep.c:3308:28: warning: unused variable ‘c’ [-Wunused-variable]
../../src/gdb/solib-frv.c: In function ‘enable_break2’:
../../src/gdb/solib-frv.c:522:10: warning: unused variable ‘bkpt_namep’ [-Wunused-variable]
../../src/gdb/solib-frv.c:521:7: warning: unused variable ‘success’ [-Wunused-variable]
../../src/gdb/solib-frv.c: In function ‘frv_fdpic_find_canonical_descriptor’:
../../src/gdb/solib-frv.c:979:13: warning: unused variable ‘exec_loadmap_addr’ [-Wunused-variable]
../../src/gdb/solib-frv.c:978:7: warning: unused variable ‘status’ [-Wunused-variable]
../../src/gdb/solib-frv.c:976:35: warning: unused variable ‘ldm’ [-Wunused-variable]
../../src/gdb/solib-svr4.c: In function ‘svr4_read_so_list’:
../../src/gdb/solib-svr4.c:1178:32: warning: unused variable ‘lmo’ [-Wunused-variable]
../../src/gdb/solib-darwin.c: In function ‘darwin_current_sos’:
../../src/gdb/solib-darwin.c:217:13: warning: unused variable ‘lm’ [-Wunused-variable]
../../src/gdb/solib-dsbt.c: In function ‘decode_loadmap’:
../../src/gdb/solib-dsbt.c:235:7: warning: unused variable ‘ext_ldmbuf_size’ [-Wunused-variable]
../../src/gdb/solib-dsbt.c: In function ‘enable_break2’:
../../src/gdb/solib-dsbt.c:845:23: warning: unused variable ‘interp_loadmap_addr’ [-Wunused-variable]
../../src/gdb/solib-dsbt.c:844:11: warning: unused variable ‘status’ [-Wunused-variable]
../../src/gdb/solib-dsbt.c:818:10: warning: unused variable ‘bkpt_namep’ [-Wunused-variable]
../../src/gdb/solib-dsbt.c:817:7: warning: unused variable ‘success’ [-Wunused-variable]
../../src/gdb/solib-dsbt.c: In function ‘dsbt_relocate_main_executable’:
../../src/gdb/solib-dsbt.c:1033:24: warning: unused variable ‘interp_addr’ [-Wunused-variable]
../../src/gdb/solib-dsbt.c:1033:13: warning: unused variable ‘exec_addr’ [-Wunused-variable]
../../src/gdb/solib-dsbt.c:1032:7: warning: unused variable ‘status’ [-Wunused-variable]
../../src/gdb/m32r-rom.c: In function ‘m32r_load’:
../../src/gdb/m32r-rom.c:126:16: warning: unused variable ‘i’ [-Wunused-variable]
../../src/gdb/m32r-rom.c:125:13: warning: unused variable ‘s’ [-Wunused-variable]
../../src/gdb/m32r-rom.c: In function ‘m32r_supply_register’:
../../src/gdb/m32r-rom.c:242:30: warning: unused variable ‘one’ [-Wunused-variable]
../../src/gdb/m32r-rom.c:242:10: warning: unused variable ‘zero’ [-Wunused-variable]
../../src/gdb/m32r-rom.c:241:18: warning: unused variable ‘psw’ [-Wunused-variable]
../../src/gdb/m32r-rom.c: In function ‘m32r_upload_command’:
../../src/gdb/m32r-rom.c:536:19: warning: unused variable ‘buffer’ [-Wunused-variable]
../../src/gdb/remote-m32r-sdi.c: In function ‘send_data’:
../../src/gdb/remote-m32r-sdi.c:184:7: warning: unused variable ‘ret’ [-Wunused-variable]
../../src/gdb/remote-m32r-sdi.c: In function ‘m32r_open’:
../../src/gdb/remote-m32r-sdi.c:370:7: warning: unused variable ‘yes’ [-Wunused-variable]
../../src/gdb/remote-m32r-sdi.c:369:10: warning: unused variable ‘n’ [-Wunused-variable]
../../src/gdb/remote-m32r-sdi.c:368:7: warning: unused variable ‘port’ [-Wunused-variable]
../../src/gdb/remote-m32r-sdi.c:366:22: warning: unused variable ‘server_addr’ [-Wunused-variable]
../../src/gdb/remote-m32r-sdi.c:365:19: warning: unused variable ‘host_ent’ [-Wunused-variable]
../../src/gdb/remote-m32r-sdi.c: In function ‘m32r_wait’:
../../src/gdb/remote-m32r-sdi.c:710:17: warning: unused variable ‘val’ [-Wunused-variable]
../../src/gdb/remote-m32r-sdi.c: In function ‘m32r_load’:
../../src/gdb/remote-m32r-sdi.c:1228:7: warning: unused variable ‘ret’ [-Wunused-variable]
../../src/gdb/remote-mips.c: In function ‘common_open’:
../../src/gdb/remote-mips.c:1540:9: warning: unused variable ‘ptype’ [-Wunused-variable]
../../src/gdb/remote-mips.c: In function ‘mips_wait’:
../../src/gdb/remote-mips.c:1831:7: warning: unused variable ‘i’ [-Wunused-variable]
../../src/gdb/remote-mips.c: In function ‘mips_fetch_registers’:
../../src/gdb/remote-mips.c:2013:19: warning: unused variable ‘byte_order’ [-Wunused-variable]
../../src/gdb/remote-mips.c: In function ‘pmon_end_download’:
../../src/gdb/remote-mips.c:3273:8: warning: unused variable ‘hexnumber’ [-Wunused-variable]
../../src/gdb/xcoffread.c: In function ‘record_minimal_symbol’:
../../src/gdb/xcoffread.c:914:28: warning: unused variable ‘args’ [-Wunused-variable]
../../src/gdb/xcoffread.c: In function ‘scan_xcoff_symtab’:
../../src/gdb/xcoffread.c:2205:7: warning: unused variable ‘past_first_source_file’ [-Wunused-variable]
../../src/gdb/alpha-mdebug-tdep.c: In function ‘alpha_mdebug_init_abi’:
../../src/gdb/alpha-mdebug-tdep.c:406:24: warning: unused variable ‘tdep’ [-Wunused-variable]
../../src/gdb/alpha-tdep.c: In function ‘_initialize_alpha_tdep’:
../../src/gdb/alpha-tdep.c:1871:28: warning: unused variable ‘c’ [-Wunused-variable]
../../src/gdb/amd64-dicos-tdep.c: In function ‘amd64_dicos_init_abi’:
../../src/gdb/amd64-dicos-tdep.c:47:24: warning: unused variable ‘tdep’ [-Wunused-variable]
../../src/gdb/amd64-darwin-tdep.c: In function ‘amd64_darwin_sigcontext_addr’:
../../src/gdb/amd64-darwin-tdep.c:87:13: warning: unused variable ‘si’ [-Wunused-variable]
../../src/gdb/ia64-tdep.c: In function ‘ia64_pseudo_register_read’:
../../src/gdb/ia64-tdep.c:1044:17: warning: unused variable ‘reg’ [-Wunused-variable]
../../src/gdb/ia64-tdep.c:1063:17: warning: unused variable ‘reg’ [-Wunused-variable]
../../src/gdb/ia64-tdep.c: In function ‘ia64_pseudo_register_write’:
../../src/gdb/ia64-tdep.c:1101:17: warning: unused variable ‘reg’ [-Wunused-variable]
../../src/gdb/ia64-tdep.c: In function ‘examine_prologue’:
../../src/gdb/ia64-tdep.c:1406:11: warning: unused variable ‘sor’ [-Wunused-variable]
../../src/gdb/ia64-tdep.c: In function ‘ia64_frame_cache’:
../../src/gdb/ia64-tdep.c:1858:7: warning: unused variable ‘i’ [-Wunused-variable]
../../src/gdb/ia64-tdep.c:1857:28: warning: unused variable ‘bsp’ [-Wunused-variable]
../../src/gdb/ia64-tdep.c:1857:23: warning: unused variable ‘sol’ [-Wunused-variable]
../../src/gdb/ia64-tdep.c:1857:18: warning: unused variable ‘sof’ [-Wunused-variable]
../../src/gdb/ia64-tdep.c: In function ‘ia64_sigtramp_frame_cache’:
../../src/gdb/ia64-tdep.c:2247:7: warning: unused variable ‘i’ [-Wunused-variable]
../../src/gdb/ia64-tdep.c:2245:13: warning: unused variable ‘addr’ [-Wunused-variable]
../../src/gdb/ia64-tdep.c: In function ‘ia64_push_dummy_call’:
../../src/gdb/ia64-tdep.c:3732:27: warning: unused variable ‘pc’ [-Wunused-variable]
../../src/gdb/ia64-tdep.c: At top level:
../../src/gdb/ia64-tdep.c:130:12: warning: ‘fp_regnum’ defined but not used [-Wunused-variable]
../../src/gdb/ia64-tdep.c:131:12: warning: ‘lr_regnum’ defined but not used [-Wunused-variable]
../../src/gdb/linux-thread-db.c: In function ‘thread_db_find_new_threads_2’:
../../src/gdb/linux-thread-db.c:1617:7: warning: unused variable ‘pid’ [-Wunused-variable]
../../src/gdb/linux-nat.c: In function ‘linux_nat_stop_lwp’:
../../src/gdb/linux-nat.c:5092:14: warning: unused variable ‘ptid’ [-Wunused-variable]
../../src/gdb/tracepoint.c: In function ‘start_tracing’:
../../src/gdb/tracepoint.c:1719:26: warning: unused variable ‘t’ [-Wunused-variable]
../../src/gdb/tracepoint.c: In function ‘tfile_fetch_registers’:
../../src/gdb/tracepoint.c:4394:18: warning: unused variable ‘mlen’ [-Wunused-variable]
../../src/gdb/tracepoint.c:4393:7: warning: unused variable ‘pos’ [-Wunused-variable]
../../src/gdb/tracepoint.c:4392:8: warning: unused variable ‘block_type’ [-Wunused-variable]
../../src/gdb/tracepoint.c: In function ‘print_one_static_tracepoint_marker’:
../../src/gdb/tracepoint.c:4775:24: warning: unused variable ‘l’ [-Wunused-variable]
../../src/gdb/remote.c: In function ‘remote_start_remote’:
../../src/gdb/remote.c:3372:24: warning: unused variable ‘inf’ [-Wunused-variable]
../../src/gdb/remote.c:3371:11: warning: unused variable ‘fake_pid_p’ [-Wunused-variable]
../../src/gdb/remote.c:3370:14: warning: unused variable ‘ptid’ [-Wunused-variable]
../../src/gdb/remote.c: In function ‘remote_add_target_side_condition’:
../../src/gdb/remote.c:7825:9: warning: unused variable ‘buf_start’ [-Wunused-variable]
../../src/gdb/remote.c:7824:9: warning: unused variable ‘pkt’ [-Wunused-variable]
../../src/gdb/remote.c: In function ‘remote_insert_breakpoint’:
../../src/gdb/remote.c:7869:30: warning: unused variable ‘cond’ [-Wunused-variable]
../../src/gdb/remote-fileio.c: In function ‘remote_fileio_func_read’:
../../src/gdb/remote-fileio.c:685:16: warning: unused variable ‘retlength’ [-Wunused-variable]
../../src/gdb/remote-fileio.c: In function ‘remote_fileio_func_fstat’:
../../src/gdb/remote-fileio.c:1136:16: warning: unused variable ‘retlength’ [-Wunused-variable]
../../src/gdb/remote-fileio.c: In function ‘remote_fileio_func_gettimeofday’:
../../src/gdb/remote-fileio.c:1215:12: warning: unused variable ‘retlength’ [-Wunused-variable]
../../src/gdb/remote-fileio.c: In function ‘remote_fileio_func_system’:
../../src/gdb/remote-fileio.c:1284:20: warning: unused variable ‘retlength’ [-Wunused-variable]
../../src/gdb/ax-gdb.c: In function ‘gen_expr’:
../../src/gdb/ax-gdb.c:2177:8: warning: unused variable ‘this_name’ [-Wunused-variable]
../../src/gdb/cli/cli-cmds.c: In function ‘alias_command’:
../../src/gdb/cli/cli-cmds.c:1428:11: warning: unused variable ‘i’ [-Wunused-variable]
../../src/gdb/cli/cli-cmds.c:1346:28: warning: unused variable ‘c’ [-Wunused-variable]
../../src/gdb/mi/mi-main.c: In function ‘get_register’:
../../src/gdb/mi/mi-main.c:1135:18: warning: unused variable ‘lval’ [-Wunused-variable]
../../src/gdb/mi/mi-main.c:1134:13: warning: unused variable ‘addr’ [-Wunused-variable]
../../src/gdb/python/python.c: In function ‘gdbpy_decode_line’:
../../src/gdb/python/python.c:559:10: warning: unused variable ‘str’ [-Wunused-variable]
../../src/gdb/python/python.c: In function ‘gdbpy_run_events’:
../../src/gdb/python/python.c:679:7: warning: unused variable ‘r’ [-Wunused-variable]
../../src/gdb/python/python.c: In function ‘_initialize_python’:
../../src/gdb/python/python.c:1152:28: warning: unused variable ‘cmd’ [-Wunused-variable]
../../src/gdb/python/python.c:1151:9: warning: unused variable ‘cmd_name’ [-Wunused-variable]
../../src/gdb/python/py-finishbreakpoint.c: In function ‘bpfinishpy_init’:
../../src/gdb/python/py-finishbreakpoint.c:157:7: warning: unused variable ‘type’ [-Wunused-variable]
../../src/gdb/python/py-finishbreakpoint.c: In function ‘bpfinishpy_out_of_scope’:
../../src/gdb/python/py-finishbreakpoint.c:316:33: warning: unused variable ‘except’ [-Wunused-variable]
../../src/gdb/python/py-finishbreakpoint.c: In function ‘bpfinishpy_detect_out_scope_cb’:
../../src/gdb/python/py-finishbreakpoint.c:339:19: warning: unused variable ‘garch’ [-Wunused-variable]
../../src/gdb/python/py-inferior.c: In function ‘find_inferior_object’:
../../src/gdb/python/py-inferior.c:183:25: warning: unused variable ‘p’ [-Wunused-variable]
../../src/gdb/python/py-inferior.c: In function ‘delete_thread_object’:
../../src/gdb/python/py-inferior.c:260:18: warning: unused variable ‘thread_obj’ [-Wunused-variable]
../../src/gdb/python/py-infthread.c: In function ‘thpy_switch’:
../../src/gdb/python/py-infthread.c:165:19: warning: unused variable ‘cleanup’ [-Wunused-variable]
../../src/gdb/python/py-type.c: In function ‘check_types_equal’:
../../src/gdb/python/py-type.c:1075:11: warning: unused variable ‘added’ [-Wunused-variable]
../../src/gdb/python/py-type.c: In function ‘typy_getitem’:
../../src/gdb/python/py-type.c:1275:33: warning: unused variable ‘except’ [-Wunused-variable]
../../src/gdb/python/py-type.c: In function ‘typy_has_key’:
../../src/gdb/python/py-type.c:1337:33: warning: unused variable ‘except’ [-Wunused-variable]
../../src/gdb/python/py-type.c: In function ‘typy_iterator_iternext’:
../../src/gdb/python/py-type.c:1434:7: warning: unused variable ‘i’ [-Wunused-variable]
../../src/gdb/c-exp.y: In function ‘parse_number’:
../../src/gdb/c-exp.y:1423:11: warning: unused variable ‘suffix_len’ [-Wunused-variable]
../../src/gdb/c-exp.y:1422:19: warning: unused variable ‘suffix’ [-Wunused-variable]
../../src/gdb/elfread.c: In function ‘elf_rel_plt_read’:
../../src/gdb/elfread.c:652:26: warning: unused variable ‘name_got_plt’ [-Wunused-variable]
../../src/gdb/elfread.c:613:12: warning: unused variable ‘u’ [-Wunused-variable]
../../src/gdb/auto-load.c: In function ‘auto_load_safe_path_vec_update’:
../../src/gdb/auto-load.c:125:17: warning: unused variable ‘dir_vec’ [-Wunused-variable]
../../src/gdb/regcache.c: In function ‘regcache_cooked_read’:
../../src/gdb/regcache.c:710:23: warning: unused variable ‘gdbarch’ [-Wunused-variable]
../../src/gdb/breakpoint.c: In function ‘set_condition_evaluation_mode’:
../../src/gdb/breakpoint.c:749:22: warning: unused variable ‘b’ [-Wunused-variable]
../../src/gdb/breakpoint.c: In function ‘parse_cond_to_aexpr’:
../../src/gdb/breakpoint.c:1948:19: warning: unused variable ‘old_chain’ [-Wunused-variable]
../../src/gdb/breakpoint.c: In function ‘catch_load_or_unload’:
../../src/gdb/breakpoint.c:7345:11: warning: unused variable ‘compiled’ [-Wunused-variable]
../../src/gdb/breakpoint.c: In function ‘print_it_catch_syscall’:
../../src/gdb/breakpoint.c:7613:9: warning: unused variable ‘syscall_id’ [-Wunused-variable]
../../src/gdb/breakpoint.c: In function ‘parse_breakpoint_sals’:
../../src/gdb/breakpoint.c:8516:9: warning: unused variable ‘addr_start’ [-Wunused-variable]
../../src/gdb/breakpoint.c: In function ‘decode_static_tracepoint_spec’:
../../src/gdb/breakpoint.c:8698:18: warning: unused variable ‘sym’ [-Wunused-variable]
../../src/gdb/breakpoint.c:8697:26: warning: unused variable ‘sal’ [-Wunused-variable]
../../src/gdb/breakpoint.c: In function ‘create_breakpoint’:
../../src/gdb/breakpoint.c:8767:7: warning: unused variable ‘i’ [-Wunused-variable]
../../src/gdb/breakpoint.c: In function ‘say_where’:
../../src/gdb/breakpoint.c:11895:18: warning: unused variable ‘uiout’ [-Wunused-variable]
../../src/gdb/breakpoint.c: In function ‘bkpt_breakpoint_hit’:
../../src/gdb/breakpoint.c:12153:22: warning: unused variable ‘b’ [-Wunused-variable]
../../src/gdb/breakpoint.c: In function ‘internal_bkpt_print_it’:
../../src/gdb/breakpoint.c:12343:18: warning: unused variable ‘uiout’ [-Wunused-variable]
../../src/gdb/breakpoint.c: In function ‘update_static_tracepoint’:
../../src/gdb/breakpoint.c:12950:7: warning: unused variable ‘i’ [-Wunused-variable]
../../src/gdb/continuations.c: In function ‘discard_my_continuations’:
../../src/gdb/continuations.c:103:24: warning: unused variable ‘continuation_ptr’ [-Wunused-variable]
../../src/gdb/eval.c: In function ‘evaluate_subexp_standard’:
../../src/gdb/eval.c:796:7: warning: unused variable ‘upper’ [-Wunused-variable]
../../src/gdb/valops.c: In function ‘search_struct_method’:
../../src/gdb/valops.c:2212:11: warning: unused variable ‘skip’ [-Wunused-variable]
../../src/gdb/valarith.c: In function ‘vector_binop’:
../../src/gdb/valarith.c:1393:52: warning: unused variable ‘result_type’ [-Wunused-variable]
../../src/gdb/value.c: In function ‘value_actual_type’:
../../src/gdb/value.c:844:17: warning: unused variable ‘target’ [-Wunused-variable]
../../src/gdb/valprint.c: In function ‘generic_val_print’:
../../src/gdb/valprint.c:332:12: warning: unused variable ‘eltlen’ [-Wunused-variable]
../../src/gdb/valprint.c:327:19: warning: unused variable ‘byte_order’ [-Wunused-variable]
../../src/gdb/printcmd.c: In function ‘map_display_numbers’:
../../src/gdb/printcmd.c:1597:23: warning: unused variable ‘tmp’ [-Wunused-variable]
../../src/gdb/printcmd.c:1597:19: warning: unused variable ‘b’ [-Wunused-variable]
../../src/gdb/printcmd.c: In function ‘undisplay_command’:
../../src/gdb/printcmd.c:1641:36: warning: unused variable ‘state’ [-Wunused-variable]
../../src/gdb/printcmd.c:1640:7: warning: unused variable ‘num’ [-Wunused-variable]
../../src/gdb/symtab.c: In function ‘compare_filenames_for_search’:
../../src/gdb/symtab.c:157:7: warning: unused variable ‘offset’ [-Wunused-variable]
../../src/gdb/symtab.c: In function ‘iterate_over_some_symtabs’:
../../src/gdb/symtab.c:197:19: warning: unused variable ‘cleanup’ [-Wunused-variable]
../../src/gdb/symtab.c: In function ‘iterate_over_symtabs’:
../../src/gdb/symtab.c:284:18: warning: unused variable ‘s’ [-Wunused-variable]
../../src/gdb/symtab.c: In function ‘find_pcs_for_symtab_line’:
../../src/gdb/symtab.c:2441:18: warning: unused variable ‘previous_function’ [-Wunused-variable]
../../src/gdb/symtab.c:2440:18: warning: unused variable ‘ix’ [-Wunused-variable]
../../src/gdb/symtab.c: In function ‘search_symbols’:
../../src/gdb/symtab.c:3258:9: warning: unused variable ‘val’ [-Wunused-variable]
../../src/gdb/symfile.c: In function ‘addrs_section_compar’:
../../src/gdb/symfile.c:547:22: warning: unused variable ‘b_idx’ [-Wunused-variable]
../../src/gdb/symfile.c:547:15: warning: unused variable ‘a_idx’ [-Wunused-variable]
../../src/gdb/symfile.c: In function ‘section_is_overlay’:
../../src/gdb/symfile.c:2979:12: warning: unused variable ‘abfd’ [-Wunused-variable]
../../src/gdb/symfile.c: In function ‘pc_in_unmapped_range’:
../../src/gdb/symfile.c:3055:12: warning: unused variable ‘abfd’ [-Wunused-variable]
../../src/gdb/symfile.c: In function ‘overlay_unmapped_address’:
../../src/gdb/symfile.c:3109:12: warning: unused variable ‘abfd’ [-Wunused-variable]
../../src/gdb/symfile.c: In function ‘overlay_mapped_address’:
../../src/gdb/symfile.c:3128:12: warning: unused variable ‘abfd’ [-Wunused-variable]
../../src/gdb/symfile.c: In function ‘simple_overlay_update_1’:
../../src/gdb/symfile.c:3518:8: warning: unused variable ‘obfd’ [-Wunused-variable]
../../src/gdb/symfile.c: In function ‘simple_overlay_update’:
../../src/gdb/symfile.c:3594:12: warning: unused variable ‘obfd’ [-Wunused-variable]
../../src/gdb/linespec.c: In function ‘find_methods’:
../../src/gdb/linespec.c:1061:11: warning: unused variable ‘name_len’ [-Wunused-variable]
../../src/gdb/linespec.c:1051:7: warning: unused variable ‘i1’ [-Wunused-variable]
../../src/gdb/linespec.c: In function ‘find_method’:
../../src/gdb/linespec.c:2674:9: warning: unused variable ‘name_iter’ [-Wunused-variable]
../../src/gdb/linespec.c: In function ‘add_matching_symbols_to_info’:
../../src/gdb/linespec.c:3389:22: warning: unused variable ‘sym’ [-Wunused-variable]
../../src/gdb/infcmd.c: In function ‘get_return_value’:
../../src/gdb/infcmd.c:1424:18: warning: unused variable ‘uiout’ [-Wunused-variable]
../../src/gdb/expprint.c: In function ‘print_subexp_standard’:
../../src/gdb/expprint.c:139:16: warning: unused variable ‘b’ [-Wunused-variable]
../../src/gdb/expprint.c: In function ‘dump_subexp_body_standard’:
../../src/gdb/expprint.c:977:8: warning: unused variable ‘elem_name’ [-Wunused-variable]
../../src/gdb/thread.c: In function ‘thread_apply_command’:
../../src/gdb/thread.c:1238:13: warning: unused variable ‘p’ [-Wunused-variable]
../../src/gdb/stack.c: In function ‘print_frame_args’:
../../src/gdb/stack.c:507:7: warning: unused variable ‘summary’ [-Wunused-variable]
../../src/gdb/stack.c:502:31: warning: unused variable ‘list_chain’ [-Wunused-variable]
../../src/gdb/parse.c: In function ‘parse_float’:
../../src/gdb/parse.c:1313:9: warning: unused variable ‘s’ [-Wunused-variable]
In file included from ../../src/gdb/observer.c:222:0:
./observer.inc: In function ‘observer_executable_changed_notification_stub’:
./observer.inc:111:41: warning: unused variable ‘args’ [-Wunused-variable]
./observer.inc: In function ‘observer_about_to_proceed_notification_stub’:
./observer.inc:458:39: warning: unused variable ‘args’ [-Wunused-variable]
../../src/gdb/objfiles.c: In function ‘filter_overlapping_sections’:
../../src/gdb/objfiles.c:1168:32: warning: unused variable ‘abfd2’ [-Wunused-variable]
../../src/gdb/objfiles.c:1167:32: warning: unused variable ‘abfd1’ [-Wunused-variable]
../../src/gdb/reverse.c: In function ‘delete_bookmark_command’:
../../src/gdb/reverse.c:219:20: warning: unused variable ‘b’ [-Wunused-variable]
../../src/gdb/coffread.c: In function ‘coff_getfilename’:
../../src/gdb/coffread.c:1309:9: warning: unused variable ‘temp’ [-Wunused-variable]
../../src/gdb/dwarf2loc.c: In function ‘call_site_find_chain_1’:
../../src/gdb/dwarf2loc.c:654:21: warning: unused variable ‘func_specific’ [-Wunused-variable]
../../src/gdb/dwarf2loc.c: In function ‘dwarf_expr_reg_to_entry_parameter’:
../../src/gdb/dwarf2loc.c:837:32: warning: unused variable ‘dwarf_block’ [-Wunused-variable]
../../src/gdb/dwarf2loc.c:836:17: warning: unused variable ‘val’ [-Wunused-variable]
../../src/gdb/dwarf2loc.c: In function ‘dwarf2_compile_expr_to_ax’:
../../src/gdb/dwarf2loc.c:2679:11: warning: unused variable ‘base_offset’ [-Wunused-variable]
../../src/gdb/dwarf2loc.c:2676:19: warning: unused variable ‘before_stack_len’ [-Wunused-variable]
../../src/gdb/dwarf2loc.c: In function ‘loclist_describe_location’:
../../src/gdb/dwarf2loc.c:3819:15: warning: unused variable ‘first’ [-Wunused-variable]
../../src/gdb/dwarf2read.c: In function ‘dw2_get_file_names’:
../../src/gdb/dwarf2read.c:2251:16: warning: unused variable ‘bytes_read’ [-Wunused-variable]
../../src/gdb/dwarf2read.c: In function ‘process_psymtab_comp_unit’:
../../src/gdb/dwarf2read.c:3440:13: warning: unused variable ‘beg_of_comp_unit’ [-Wunused-variable]
../../src/gdb/dwarf2read.c: In function ‘psymtab_to_symtab_1’:
../../src/gdb/dwarf2read.c:4659:19: warning: unused variable ‘back_to’ [-Wunused-variable]
../../src/gdb/dwarf2read.c: In function ‘read_call_site_scope’:
../../src/gdb/dwarf2read.c:6385:36: warning: unused variable ‘dlbaton’ [-Wunused-variable]
../../src/gdb/dwarf2read.c: In function ‘dwarf2_add_typedef’:
../../src/gdb/dwarf2read.c:7187:9: warning: unused variable ‘fieldname’ [-Wunused-variable]
../../src/gdb/dwarf2read.c:7185:21: warning: unused variable ‘attr’ [-Wunused-variable]
../../src/gdb/dwarf2read.c:7183:19: warning: unused variable ‘objfile’ [-Wunused-variable]
../../src/gdb/dwarf2read.c: In function ‘dwarf2_const_value’:
../../src/gdb/dwarf2read.c:12184:26: warning: unused variable ‘cu_header’ [-Wunused-variable]
../../src/gdb/dwarf2read.c: In function ‘lookup_die_type’:
../../src/gdb/dwarf2read.c:12316:25: warning: unused variable ‘sig_cu’ [-Wunused-variable]
../../src/gdb/dwarf2read.c: In function ‘add_address_entry_worker’:
../../src/gdb/dwarf2read.c:16819:10: warning: unused variable ‘slot’ [-Wunused-variable]
../../src/gdb/dwarf2read.c:16818:15: warning: unused variable ‘cu_index’ [-Wunused-variable]
../../src/gdb/dwarf2read.c: In function ‘write_psymtabs_to_index’:
../../src/gdb/dwarf2read.c:16998:8: warning: unused variable ‘buf’ [-Wunused-variable]
../../src/gdb/dwarf2expr.c: In function ‘execute_stack_op’:
../../src/gdb/dwarf2expr.c:865:20: warning: unused variable ‘die’ [-Wunused-variable]
../../src/gdb/dwarf2-frame.c: In function ‘dwarf2_compile_cfa_to_ax’:
../../src/gdb/dwarf2-frame.c:894:26: warning: unused variable ‘cfa’ [-Wunused-variable]
../../src/gdb/dwarf2-frame.c:891:13: warning: unused variable ‘num_regs’ [-Wunused-variable]
../../src/gdb/dwarf2-frame-tailcall.c: In function ‘pretend_pc’:
../../src/gdb/dwarf2-frame-tailcall.c:239:7: warning: unused variable ‘caller_no’ [-Wunused-variable]
../../src/gdb/dwarf2-frame-tailcall.c: In function ‘dwarf2_tailcall_sniffer_first’:
../../src/gdb/dwarf2-frame-tailcall.c:373:22: warning: unused variable ‘fi’ [-Wunused-variable]
../../src/gdb/dwarf2-frame-tailcall.c:370:22: warning: unused variable ‘pc’ [-Wunused-variable]
../../src/gdb/c-lang.c: In function ‘c_printstr’:
../../src/gdb/c-lang.c:210:7: warning: unused variable ‘need_escape’ [-Wunused-variable]
../../src/gdb/c-lang.c:209:7: warning: unused variable ‘finished’ [-Wunused-variable]
../../src/gdb/c-lang.c:208:26: warning: unused variable ‘iter’ [-Wunused-variable]
../../src/gdb/c-lang.c:207:19: warning: unused variable ‘cleanup’ [-Wunused-variable]
../../src/gdb/c-lang.c:206:29: warning: unused variable ‘output’ [-Wunused-variable]
../../src/gdb/c-lang.c:206:18: warning: unused variable ‘wchar_buf’ [-Wunused-variable]
../../src/gdb/c-lang.c:205:7: warning: unused variable ‘width’ [-Wunused-variable]
../../src/gdb/c-lang.c:204:7: warning: unused variable ‘need_comma’ [-Wunused-variable]
../../src/gdb/c-lang.c:203:7: warning: unused variable ‘in_quotes’ [-Wunused-variable]
../../src/gdb/c-lang.c:202:16: warning: unused variable ‘things_printed’ [-Wunused-variable]
../../src/gdb/c-lang.c:201:16: warning: unused variable ‘i’ [-Wunused-variable]
../../src/gdb/c-lang.c:200:19: warning: unused variable ‘byte_order’ [-Wunused-variable]
../../src/gdb/objc-lang.c: In function ‘find_methods’:
../../src/gdb/objc-lang.c:1154:20: warning: unused variable ‘gdbarch’ [-Wunused-variable]
../../src/gdb/ada-lang.c: In function ‘wild_match’:
../../src/gdb/ada-lang.c:5473:19: warning: unused variable ‘n’ [-Wunused-variable]
../../src/gdb/ada-lang.c: In function ‘ada_exception_support_info_sniffer’:
../../src/gdb/ada-lang.c:10824:18: warning: unused variable ‘sym’ [-Wunused-variable]
../../src/gdb/ada-tasks.c: In function ‘ada_tasks_inferior_data_sniffer’:
../../src/gdb/ada-tasks.c:850:15: warning: unused variable ‘name’ [-Wunused-variable]
../../src/gdb/ada-varobj.c: In function ‘ada_varobj_get_array_number_of_children’:
../../src/gdb/ada-varobj.c:230:7: warning: unused variable ‘len’ [-Wunused-variable]
../../src/gdb/jv-valprint.c: In function ‘java_val_print’:
../../src/gdb/jv-valprint.c:491:16: warning: unused variable ‘i’ [-Wunused-variable]
../../src/gdb/varobj.c: In function ‘varobj_update’:
../../src/gdb/varobj.c:1968:7: warning: unused variable ‘changed’ [-Wunused-variable]
../../src/gdb/p-valprint.c: In function ‘pascal_val_print’:
../../src/gdb/p-valprint.c:75:8: warning: unused variable ‘val’ [-Wunused-variable]
../../src/gdb/c-valprint.c: In function ‘c_val_print’:
../../src/gdb/c-valprint.c:148:8: warning: unused variable ‘val’ [-Wunused-variable]
../../src/gdb/m2-valprint.c: In function ‘m2_val_print’:
../../src/gdb/m2-valprint.c:317:8: warning: unused variable ‘val’ [-Wunused-variable]
../../src/gdb/m2-valprint.c:312:19: warning: unused variable ‘gdbarch’ [-Wunused-variable]
../../src/gdb/f-valprint.c: In function ‘f_val_print’:
../../src/gdb/f-valprint.c:270:8: warning: unused variable ‘val’ [-Wunused-variable]
../../src/gdb/f-valprint.c:267:19: warning: unused variable ‘byte_order’ [-Wunused-variable]
../../src/gdb/gnu-v3-abi.c: In function ‘compute_vtable_size’:
../../src/gdb/gnu-v3-abi.c:801:13: warning: unused variable ‘addr’ [-Wunused-variable]
../../src/gdb/cp-namespace.c: In function ‘cp_lookup_symbol_imports_or_template’:
../../src/gdb/cp-namespace.c:477:30: warning: unused variable ‘cps’ [-Wunused-variable]
../../src/gdb/cp-namespace.c:476:11: warning: unused variable ‘i’ [-Wunused-variable]
../../src/gdb/cp-support.c: In function ‘make_symbol_overload_list_qualified’:
../../src/gdb/cp-support.c:1330:28: warning: unused variable ‘dict’ [-Wunused-variable]
../../src/gdb/cp-support.c:1329:24: warning: unused variable ‘iter’ [-Wunused-variable]
../../src/gdb/cp-support.c:1325:18: warning: unused variable ‘sym’ [-Wunused-variable]
../../src/gdb/solib-target.c: In function ‘solib_target_relocate_section_addresses’:
../../src/gdb/solib-target.c:328:7: warning: unused variable ‘flags’ [-Wunused-variable]
../../src/gdb/gcore.c: In function ‘objfile_find_memory_regions’:
../../src/gdb/gcore.c:475:12: warning: unused variable ‘ibfd’ [-Wunused-variable]
../../src/gdb/jit.c: In function ‘jit_reader_load_command’:
../../src/gdb/jit.c:203:7: warning: unused variable ‘len’ [-Wunused-variable]
../../src/gdb/jit.c: In function ‘add_objfile_entry’:
../../src/gdb/jit.c:283:14: warning: unused variable ‘entry_addr_ptr’ [-Wunused-variable]
../../src/gdb/jit.c: In function ‘jit_reader_try_read_symtab’:
../../src/gdb/jit.c:777:26: warning: unused variable ‘i’ [-Wunused-variable]
../../src/gdb/jit.c: In function ‘jit_register_code’:
../../src/gdb/jit.c:908:29: warning: unused variable ‘inf_data’ [-Wunused-variable]
../../src/gdb/jit.c:907:31: warning: unused variable ‘b’ [-Wunused-variable]
../../src/gdb/jit.c:906:7: warning: unused variable ‘i’ [-Wunused-variable]
../../src/gdb/jit.c: In function ‘jit_find_objf_with_entry_addr’:
../../src/gdb/jit.c:938:14: warning: unused variable ‘objf_entry_addr’ [-Wunused-variable]
../../src/gdb/jit.c: In function ‘jit_frame_sniffer’:
../../src/gdb/jit.c:1098:26: warning: unused variable ‘iter’ [-Wunused-variable]
../../src/gdb/jit.c: In function ‘jit_inferior_init’:
../../src/gdb/jit.c:1240:28: warning: unused variable ‘objf_data’ [-Wunused-variable]
../../src/gdb/skip.c: In function ‘skip_function_command’:
../../src/gdb/skip.c:160:11: warning: unused variable ‘pending’ [-Wunused-variable]
../../src/gdb/skip.c:159:11: warning: unused variable ‘i’ [-Wunused-variable]
i386-linux.c updated.
i386-avx-linux.c updated.
amd64-linux.c updated.
i386-mmx-linux.c updated.
x32-linux.c updated.
x32-avx-linux.c updated.
amd64-avx-linux.c updated.

Re: Switch -Wunused-variable on?

[Michael Eager]

On 04/23/2012 10:25 AM, Pedro Alves wrote:
> On 04/23/2012 04:49 PM, Sergio Durigan Junior wrote:
>
>> On Monday, April 23 2012, Tom Tromey wrote:
>>
>>> I'd say that the best approach would be to make the declarations
>>> conditional in the same way that the uses are.
>>>
>>> If there are a lot of problem cases with this approach, or if it makes
>>> the code too ugly, then maybe it would be better just not to use this
>>> flag.
>>
>> Ok, that makes sense to me.  Since there are tons of places with unused
>> variables in the code, I will take some time to look and convert the
>> proper places to use this solution.
>
>
> I don't think there'll be that many such cases.
>
> In case it helps, here's how I peek at this sort of thing:
>
> $ .....configure --enable-targets=all
> $ make -j8
> $ cd gdb
> $ make clean
> $ make WERROR_CFLAGS="-Wunused-variable" -j8 2>&1 1>/dev/null | tee warnings.txt
> ...
>
> There aren't _that_ many warnings:
>
> $ grep warning warnings.txt | wc -l
> 512
>
> (This doesn't catch files only built on other hosts, of course.)
>
> And the worse offenders account for a lot of the instances:
>
> $ grep warning warnings.txt | sed 's/\.c.*/\.c/g' | sort | uniq -c | sort -nr
>       50 ../../src/gdb/arm-tdep.c
>       15 ../../src/gdb/cris-tdep.c
>       13 ../../src/gdb/ia64-tdep.c
>       12 ../../src/gdb/dwarf2read.c
>       12 ../../src/gdb/c-lang.c
>       12 ../../src/gdb/breakpoint.c
>       11 ../../src/gdb/mips-tdep.c
>       11 ../../src/gdb/m32c-tdep.c
>       10 ../../src/gdb/spu-tdep.c
>       10 ../../src/gdb/lm32-tdep.c
>       10 ../../src/gdb/hppa-tdep.c

> ../../src/gdb/microblaze-tdep.c: In function ‘microblaze_alloc_frame_cache’:
> ../../src/gdb/microblaze-tdep.c:176:7: warning: unused variable ‘i’ [-Wunused-variable]
> ../../src/gdb/microblaze-tdep.c: In function ‘microblaze_analyze_prologue’:
> ../../src/gdb/microblaze-tdep.c:234:7: warning: unused variable ‘rn’ [-Wunused-variable]
> ../../src/gdb/microblaze-tdep.c: In function ‘microblaze_frame_cache’:
> ../../src/gdb/microblaze-tdep.c:464:23: warning: unused variable ‘fp’ [-Wunused-variable]
> ../../src/gdb/microblaze-tdep.c:464:19: warning: unused variable ‘pc’ [-Wunused-variable]
> ../../src/gdb/microblaze-linux-tdep.c: In function ‘microblaze_linux_sigtramp_cache’:
> ../../src/gdb/microblaze-linux-tdep.c:76:24: warning: unused variable ‘tdep’ [-Wunused-variable]
> ../../src/gdb/microblaze-linux-tdep.c: In function ‘microblaze_linux_init_abi’:
> ../../src/gdb/microblaze-linux-tdep.c:124:24: warning: unused variable ‘tdep’ [-Wunused-variable]

I'll clean up the MicroBlaze target files when I have a moment.

-- 
Michael Eager	 eager@eagercon.com
1960 Park Blvd., Palo Alto, CA 94306  650-325-8077

Re: Switch -Wunused-variable on?

[Maciej W. Rozycki]

On Mon, 23 Apr 2012, Tom Tromey wrote:

> I'd say that the best approach would be to make the declarations
> conditional in the same way that the uses are.
> 
> If there are a lot of problem cases with this approach, or if it makes
> the code too ugly, then maybe it would be better just not to use this
> flag.

 That IMHO just asks to refrain from interspersing function bodies with 
preprocessor conditionals.  Lots if not all conditional stuff can be 
factored out to headers or definitions elsewhere.  Linux the kernel for 
one has had such a policy for years now.

 Example.  Do not write this:

int target_frobnicate (int count);

#ifdef TARGET_NEEDS_HARD_FROBNICATION
int target_frobnicate_harder (int count, int status);
#endif

int
handle_frobnication (int count)
{
  int status;
  int i;

  status = target_frobnicate (count);
#ifdef TARGET_NEEDS_HARD_FROBNICATION
  for (i = 0; i < count; i++)
    status = target_frobnicate_harder (i, status);
#endif

  return status;
}

Instead write this:

int target_frobnicate (int count);

#ifdef TARGET_NEEDS_HARD_FROBNICATION
int target_frobnicate_harder (int count, int status);
#define target_needs_hard_frobnication 1
#else
#define target_needs_hard_frobnication 0
endif

int
handle_frobnication (int count)
{
  int status;
  int i;

  status = target_frobnicate (count);
  if (target_needs_hard_frobnication)
    for (i = 0; i < count; i++)
      status = target_frobnicate_harder (i, status);

  return status;
}

-- "i" is now live in all cases and also IMHO handle_frobnication is more 
readable.

  Maciej

Re: Switch -Wunused-variable on?

[Andreas Schwab]

"Maciej W. Rozycki" <macro@codesourcery.com> writes:

> Instead write this:
>
> int target_frobnicate (int count);
>
> #ifdef TARGET_NEEDS_HARD_FROBNICATION
> int target_frobnicate_harder (int count, int status);
> #define target_needs_hard_frobnication 1
> #else
> #define target_needs_hard_frobnication 0
> endif
>
> int
> handle_frobnication (int count)
> {
>   int status;
>   int i;
>
>   status = target_frobnicate (count);
>   if (target_needs_hard_frobnication)
>     for (i = 0; i < count; i++)
>       status = target_frobnicate_harder (i, status);
>
>   return status;
> }
>
> -- "i" is now live in all cases and also IMHO handle_frobnication is more 
> readable.

It also requires an unconditional declaration and definition of
target_frobnicate_harder.

Andreas.

-- 
Andreas Schwab, schwab@linux-m68k.org
GPG Key fingerprint = 58CA 54C7 6D53 942B 1756  01D3 44D5 214B 8276 4ED5
"And now for something completely different."

Re: Switch -Wunused-variable on?

[Maciej W. Rozycki]

On Wed, 25 Apr 2012, Andreas Schwab wrote:

> > -- "i" is now live in all cases and also IMHO handle_frobnication is more 
> > readable.
> 
> It also requires an unconditional declaration and definition of
> target_frobnicate_harder.

 Yes, but do you see at as advantage or disadvantage?

 Please note that my example can be easily adapted to a three-way choice 
where the value of target_frobnicate_harder can only be determined at the 
run time, e.g.:

#ifdef TARGET_NEEDS_HARD_FROBNICATION
#define target_needs_hard_frobnication 1
#else if defined TARGET_SUPPORTS_HARD_FROBNICATION
int target_needs_hard_frobnication;
#else
#define target_needs_hard_frobnication 0
endif

and then you still have all the conditional bits localised here and don't 
have to modify handle_frobnication at at all. :)

  Maciej

Re: Switch -Wunused-variable on?

[Andreas Schwab]

"Maciej W. Rozycki" <macro@codesourcery.com> writes:

> On Wed, 25 Apr 2012, Andreas Schwab wrote:
>
>> > -- "i" is now live in all cases and also IMHO handle_frobnication is more 
>> > readable.
>> 
>> It also requires an unconditional declaration and definition of
>> target_frobnicate_harder.
>
>  Yes, but do you see at as advantage or disadvantage?

It makes your example non-working as written due to the missing
declaration.

Andreas.

-- 
Andreas Schwab, schwab@linux-m68k.org
GPG Key fingerprint = 58CA 54C7 6D53 942B 1756  01D3 44D5 214B 8276 4ED5
"And now for something completely different."

Re: Switch -Wunused-variable on?

[Maciej W. Rozycki]

On Wed, 25 Apr 2012, Andreas Schwab wrote:

> >> It also requires an unconditional declaration and definition of
> >> target_frobnicate_harder.
> >
> >  Yes, but do you see at as advantage or disadvantage?
> 
> It makes your example non-working as written due to the missing
> declaration.

 The declaration should be moved outside the conditional, I've overlooked 
that, sorry.  If target_frobnicate_harder is external then the prototype 
will be in a header anyway.

 And such an unconditional declaration is actually good for conditional 
pieces that are rarely enabled -- people making bulk changes will still 
receive the usual diagnostics about this function's argument type 
compatibility, etc. even if this piece of code is actually not used for 
the configuration they chose, as long as this source file is compiled at 
all.

 Then you don't actually need a definition, because the call will be 
optimised away (together with the whole enclosing "if (0)" block) by the 
compiler.  There's nothing wrong with having a declaration and no 
definition for an entity that's never eventually used.

  Maciej

Re: Switch -Wunused-variable on?

[Andreas Schwab]

"Maciej W. Rozycki" <macro@codesourcery.com> writes:

>  Then you don't actually need a definition, because the call will be 
> optimised away

You still need to support unoptimized builds.

Andreas.

-- 
Andreas Schwab, schwab@linux-m68k.org
GPG Key fingerprint = 58CA 54C7 6D53 942B 1756  01D3 44D5 214B 8276 4ED5
"And now for something completely different."

Re: Switch -Wunused-variable on?

[Sterling Augustine]

On Wed, Apr 25, 2012 at 11:13 AM, Andreas Schwab <schwab@linux-m68k.org> wrote:
> "Maciej W. Rozycki" <macro@codesourcery.com> writes:
>
>>  Then you don't actually need a definition, because the call will be
>> optimised away
>
> You still need to support unoptimized builds.

GCC eliminates the unreachable block of an "if (0)" even without
optimization. Many other compilers don't. It is an ongoing source of
complications for getting compilers other than GCC to build the linux
kernel which uses this approach extensively.

If the project cares about being able to build with compilers other
than GCC, then it should consider this approach as carefully as it
considers using any other GCC extension.

Sterling

Re: Switch -Wunused-variable on?

[Maciej W. Rozycki]

On Wed, 25 Apr 2012, Sterling Augustine wrote:

> >>  Then you don't actually need a definition, because the call will be
> >> optimised away
> >
> > You still need to support unoptimized builds.
> 
> GCC eliminates the unreachable block of an "if (0)" even without
> optimization. Many other compilers don't. It is an ongoing source of
> complications for getting compilers other than GCC to build the linux
> kernel which uses this approach extensively.

 That sounds silly, I didn't know that.  In this case a "static inline" 
dummy function would have to be provided indeed that makes the solution a 
bit less elegant then I would like it to be.  Still probably better than a 
maze of #ifdefs scattered throughout code, but YMMV.

  Maciej