From 2b804145796e948fa4c025c07eb201e700281e6b Mon Sep 17 00:00:00 2001 From: Alan Modra Date: Wed, 5 Oct 2016 16:52:24 +1030 Subject: -Wimplicit-fallthrough error fixes Well, not all are errors, but a little more substantive than just fiddling with comments. bfd/ * coffcode.h (coff_slurp_symbol_table): Revert accidental commit made 2015-01-08. * elf32-nds32.c (nds32_elf_grok_psinfo): Add missing break. * reloc.c (bfd_default_reloc_type_lookup): Add missing breaks. opcodes/ * arc-ext.c (create_map): Add missing break. * msp430-decode.opc (encode_as): Likewise. * msp430-decode.c: Regenerate. binutils/ * coffdump.c (dump_coff_where): Add missing break. * stabs.c (stab_xcoff_builtin_type): Likewise. gas/ * config/tc-arc.c (find_opcode_match): Add missing break. * config/tc-i960.c (get_cdisp): Likewise. * config/tc-metag.c (parse_swap, md_apply_fix): Likewise. * config/tc-mt.c (md_parse_option): Likewise. * config/tc-nds32.c (nds32_apply_fix): Likewise. * config/tc-hppa.c (pa_ip): Assert rather than testing last condition of multiple if statements. * config/tc-s390.c (s390_exp_compare): Return 0 on error. * config/tc-tic4x.c (tic4x_operand_parse): Add as_bad and break out of case rather than falling into next case. Formatting. ld/ * plugin.c (asymbol_from_plugin_symbol): Avoid compiler warning by adding return. --- binutils/ChangeLog | 5 +++++ binutils/coffdump.c | 1 + binutils/stabs.c | 1 + 3 files changed, 7 insertions(+) (limited to 'binutils') diff --git a/binutils/ChangeLog b/binutils/ChangeLog index c40ceda..bef5ec0 100644 --- a/binutils/ChangeLog +++ b/binutils/ChangeLog @@ -1,3 +1,8 @@ +2016-10-06 Alan Modra + + * coffdump.c (dump_coff_where): Add missing break. + * stabs.c (stab_xcoff_builtin_type): Likewise. + 2016-09-29 Alan Modra * readelf.c (process_arch_specific): Call process_power_specific diff --git a/binutils/coffdump.c b/binutils/coffdump.c index e9aa76f..6404458 100644 --- a/binutils/coffdump.c +++ b/binutils/coffdump.c @@ -267,6 +267,7 @@ dump_coff_where (struct coff_where *p) break; case coff_where_strtag: printf ("STRTAG"); + break; case coff_where_entag: printf ("ENTAG"); break; diff --git a/binutils/stabs.c b/binutils/stabs.c index aebde7a..d209094 100644 --- a/binutils/stabs.c +++ b/binutils/stabs.c @@ -3432,6 +3432,7 @@ stab_xcoff_builtin_type (void *dhandle, struct stab_handle *info, case 9: name = "unsigned"; rettype = debug_make_int_type (dhandle, 4, TRUE); + break; case 10: name = "unsigned long"; rettype = debug_make_int_type (dhandle, 4, TRUE); -- cgit v1.1