-Wimplicit-fallthrough warning fixes
authorAlan Modra <amodra@gmail.com>
Wed, 5 Oct 2016 07:47:02 +0000 (18:17 +1030)
committerAlan Modra <amodra@gmail.com>
Wed, 5 Oct 2016 23:43:15 +0000 (10:13 +1030)
Comment changes.

bfd/
* coff-h8300.c: Spell fall through comments consistently.
* coffgen.c: Likewise.
* elf32-hppa.c: Likewise.
* elf32-ppc.c: Likewise.
* elf32-score.c: Likewise.
* elf32-score7.c: Likewise.
* elf64-ppc.c: Likewise.
* elfxx-aarch64.c: Likewise.
* elfxx-mips.c: Likewise.
* cpu-ns32k.c: Add missing fall through comments.
* elf-m10300.c: Likewise.
* elf32-arm.c: Likewise.
* elf32-avr.c: Likewise.
* elf32-bfin.c: Likewise.
* elf32-frv.c: Likewise.
* elf32-i386.c: Likewise.
* elf32-microblaze.c: Likewise.
* elf32-nds32.c: Likewise.
* elf32-ppc.c: Likewise.
* elf32-rl78.c: Likewise.
* elf32-rx.c: Likewise.
* elf32-s390.c: Likewise.
* elf32-sh.c: Likewise.
* elf32-tic6x.c: Likewise.
* elf64-ia64-vms.c: Likewise.
* elf64-ppc.c: Likewise.
* elf64-s390.c: Likewise.
* elf64-x86-64.c: Likewise.
* elflink.c: Likewise.
* elfnn-aarch64.c: Likewise.
* elfnn-ia64.c: Likewise.
* ieee.c: Likewise.
* oasys.c: Likewise.
* pdp11.c: Likewise.
* srec.c: Likewise.
* versados.c: Likewise.
opcodes/
* aarch64-opc.c: Spell fall through comments consistently.
* i386-dis.c: Likewise.
* aarch64-dis.c: Add missing fall through comments.
* aarch64-opc.c: Likewise.
* arc-dis.c: Likewise.
* arm-dis.c: Likewise.
* i386-dis.c: Likewise.
* m68k-dis.c: Likewise.
* mep-asm.c: Likewise.
* ns32k-dis.c: Likewise.
* sh-dis.c: Likewise.
* tic4x-dis.c: Likewise.
* tic6x-dis.c: Likewise.
* vax-dis.c: Likewise.
binutils/
* dlltool.c: Spell fall through comments consistently.
* objcopy.c: Likewise.
* readelf.c: Likewise.
* dwarf.c: Add missing fall through comments.
* elfcomm.c: Likewise.
* sysinfo.y: Likewise.
* readelf.c: Likewise.  Also remove extraneous comments.
gas/
* app.c: Add missing fall through comments.
* dw2gencfi.c: Likewise.
* expr.c: Likewise.
* config/tc-alpha.c: Likewise.
* config/tc-arc.c: Likewise.
* config/tc-arm.c: Likewise.
* config/tc-cr16.c: Likewise.
* config/tc-crx.c: Likewise.
* config/tc-dlx.c: Likewise.
* config/tc-h8300.c: Likewise.
* config/tc-hppa.c: Likewise.
* config/tc-i370.c: Likewise.
* config/tc-i386.c: Likewise.
* config/tc-i960.c: Likewise.
* config/tc-ia64.c: Likewise.
* config/tc-m68hc11.c: Likewise.
* config/tc-m68k.c: Likewise.
* config/tc-mep.c: Likewise.
* config/tc-metag.c: Likewise.
* config/tc-microblaze.c: Likewise.
* config/tc-mips.c: Likewise.
* config/tc-ns32k.c: Likewise.
* config/tc-rx.c: Likewise.
* config/tc-score.c: Likewise.
* config/tc-score7.c: Likewise.
* config/tc-sh.c: Likewise.
* config/tc-tic4x.c: Likewise.
* config/tc-vax.c: Likewise.
* config/tc-xstormy16.c: Likewise.
* config/tc-z80.c: Likewise.
* config/tc-z8k.c: Likewise.
* config/obj-elf.c: Likewise.
* config/tc-i386.c: Likewise.
* depend.c: Spell fall through comments consistently.
* config/tc-arm.c: Likewise.
* config/tc-d10v.c: Likewise.
* config/tc-i960.c: Likewise.
* config/tc-ia64.c: Likewise.
* config/tc-m68k.c: Likewise.
* config/tc-mcore.c: Likewise.
* config/tc-mep.c: Likewise.
* config/tc-ns32k.c: Likewise.
* config/tc-visium.c: Likewise.
* config/tc-xstormy16.c: Likewise.
* config/tc-z8k.c: Likewise.
gprof/
* gprof.c: Add missing fall through comments.
ld/
* lexsup.c: Spell fall through comments consistently and add
missing fall through comments.

96 files changed:
bfd/ChangeLog
bfd/coff-h8300.c
bfd/coffgen.c
bfd/cpu-ns32k.c
bfd/elf-m10300.c
bfd/elf32-arm.c
bfd/elf32-avr.c
bfd/elf32-bfin.c
bfd/elf32-frv.c
bfd/elf32-hppa.c
bfd/elf32-i386.c
bfd/elf32-microblaze.c
bfd/elf32-nds32.c
bfd/elf32-ppc.c
bfd/elf32-rl78.c
bfd/elf32-rx.c
bfd/elf32-s390.c
bfd/elf32-score.c
bfd/elf32-score7.c
bfd/elf32-sh.c
bfd/elf32-tic6x.c
bfd/elf64-ia64-vms.c
bfd/elf64-ppc.c
bfd/elf64-s390.c
bfd/elf64-x86-64.c
bfd/elflink.c
bfd/elfnn-aarch64.c
bfd/elfnn-ia64.c
bfd/elfxx-aarch64.c
bfd/elfxx-mips.c
bfd/ieee.c
bfd/oasys.c
bfd/pdp11.c
bfd/srec.c
bfd/versados.c
binutils/ChangeLog
binutils/dlltool.c
binutils/dwarf.c
binutils/elfcomm.c
binutils/objcopy.c
binutils/readelf.c
binutils/sysinfo.y
gas/ChangeLog
gas/app.c
gas/config/obj-elf.c
gas/config/tc-alpha.c
gas/config/tc-arc.c
gas/config/tc-arm.c
gas/config/tc-cr16.c
gas/config/tc-crx.c
gas/config/tc-d10v.c
gas/config/tc-dlx.c
gas/config/tc-h8300.c
gas/config/tc-hppa.c
gas/config/tc-i370.c
gas/config/tc-i386.c
gas/config/tc-i960.c
gas/config/tc-ia64.c
gas/config/tc-m68hc11.c
gas/config/tc-m68k.c
gas/config/tc-mcore.c
gas/config/tc-mep.c
gas/config/tc-metag.c
gas/config/tc-microblaze.c
gas/config/tc-mips.c
gas/config/tc-ns32k.c
gas/config/tc-rx.c
gas/config/tc-score.c
gas/config/tc-score7.c
gas/config/tc-sh.c
gas/config/tc-tic4x.c
gas/config/tc-vax.c
gas/config/tc-visium.c
gas/config/tc-xstormy16.c
gas/config/tc-z80.c
gas/config/tc-z8k.c
gas/depend.c
gas/dw2gencfi.c
gas/expr.c
gprof/ChangeLog
gprof/gprof.c
ld/ChangeLog
ld/lexsup.c
opcodes/ChangeLog
opcodes/aarch64-dis.c
opcodes/aarch64-opc.c
opcodes/arc-dis.c
opcodes/arm-dis.c
opcodes/i386-dis.c
opcodes/m68k-dis.c
opcodes/mep-asm.c
opcodes/ns32k-dis.c
opcodes/sh-dis.c
opcodes/tic4x-dis.c
opcodes/tic6x-dis.c
opcodes/vax-dis.c

index 11381ea1047dac35c25f069c015668a84e543c5a..a276240abe23aa8d8bba183ad25f0c7e1d4ae822 100644 (file)
@@ -1,3 +1,42 @@
+2016-10-06  Alan Modra  <amodra@gmail.com>
+
+       * coff-h8300.c: Spell fall through comments consistently.
+       * coffgen.c: Likewise.
+       * elf32-hppa.c: Likewise.
+       * elf32-ppc.c: Likewise.
+       * elf32-score.c: Likewise.
+       * elf32-score7.c: Likewise.
+       * elf64-ppc.c: Likewise.
+       * elfxx-aarch64.c: Likewise.
+       * elfxx-mips.c: Likewise.
+       * cpu-ns32k.c: Add missing fall through comments.
+       * elf-m10300.c: Likewise.
+       * elf32-arm.c: Likewise.
+       * elf32-avr.c: Likewise.
+       * elf32-bfin.c: Likewise.
+       * elf32-frv.c: Likewise.
+       * elf32-i386.c: Likewise.
+       * elf32-microblaze.c: Likewise.
+       * elf32-nds32.c: Likewise.
+       * elf32-ppc.c: Likewise.
+       * elf32-rl78.c: Likewise.
+       * elf32-rx.c: Likewise.
+       * elf32-s390.c: Likewise.
+       * elf32-sh.c: Likewise.
+       * elf32-tic6x.c: Likewise.
+       * elf64-ia64-vms.c: Likewise.
+       * elf64-ppc.c: Likewise.
+       * elf64-s390.c: Likewise.
+       * elf64-x86-64.c: Likewise.
+       * elflink.c: Likewise.
+       * elfnn-aarch64.c: Likewise.
+       * elfnn-ia64.c: Likewise.
+       * ieee.c: Likewise.
+       * oasys.c: Likewise.
+       * pdp11.c: Likewise.
+       * srec.c: Likewise.
+       * versados.c: Likewise.
+
 2016-10-06  Alan Modra  <amodra@gmail.com>
 
        * coffcode.h (coff_slurp_symbol_table): Revert accidental commit
index 3e4722c0baf0d5e4a3dd694a4c19f2ee0a4ed68a..a6877088a944dbc7fe8e58db7e1230c52950b451 100644 (file)
@@ -579,9 +579,7 @@ h8300_reloc16_estimate (bfd *abfd, asection *input_section, arelent *reloc,
          /* Done with this reloc.  */
          break;
        }
-
-      /* FALLTHROUGH and try to turn the 24-/32-bit reloc into a 16-bit
-        reloc.  */
+      /* Fall through.  */
 
     /* This is a 24-/32-bit absolute address in a mov insn, which can
        become an 16-bit absolute address if it's in the right range.  */
index 216102ebee76004f859d02170ac90ee965aead51..1dc77a4cb18d2d92030ebb13510f76a591080f62 100644 (file)
@@ -2134,7 +2134,7 @@ coff_print_symbol (bfd *abfd,
                                 auxp->u.auxent.x_scn.x_comdat);
                      break;
                    }
-                   /* Otherwise fall through.  */
+                 /* Fall through.  */
                case C_EXT:
                case C_AIX_WEAKEXT:
                  if (ISFCN (combined->u.syment.n_type))
@@ -2154,7 +2154,7 @@ coff_print_symbol (bfd *abfd,
                               llnos, next);
                      break;
                    }
-                 /* Otherwise fall through.  */
+                 /* Fall through.  */
                default:
                  fprintf (file, "AUX lnno %d size 0x%x tagndx %ld",
                           auxp->u.auxent.x_sym.x_misc.x_lnsz.x_lnno,
index b11318c16c10b81668831e9b01806c2160a4317e..98f22e1881c33051a49bf1e20a7e07f910058b69 100644 (file)
@@ -106,8 +106,10 @@ _bfd_ns32k_get_immediate (bfd_byte *buffer, int size)
     case 4:
       value = (value << 8) | (*buffer++ & 0xff);
       value = (value << 8) | (*buffer++ & 0xff);
+      /* Fall through.  */
     case 2:
       value = (value << 8) | (*buffer++ & 0xff);
+      /* Fall through.  */
     case 1:
       value = (value << 8) | (*buffer++ & 0xff);
       break;
@@ -126,8 +128,10 @@ _bfd_ns32k_put_immediate (bfd_vma value, bfd_byte *buffer, int size)
     case 4:
       *buffer-- = (value & 0xff); value >>= 8;
       *buffer-- = (value & 0xff); value >>= 8;
+      /* Fall through.  */
     case 2:
       *buffer-- = (value & 0xff); value >>= 8;
+      /* Fall through.  */
     case 1:
       *buffer-- = (value & 0xff); value >>= 8;
     }
index 9b44c0add19320bf5cf7faabc7abceb091141bca..25f1668634779cf19e593941ba0a983113b951d8 100644 (file)
@@ -1470,6 +1470,7 @@ mn10300_elf_final_link_relocate (reloc_howto_type *howto,
          && h != NULL
          && ! SYMBOL_REFERENCES_LOCAL (info, h))
        return bfd_reloc_dangerous;
+      /* Fall through.  */
     case R_MN10300_GOT32:
       /* Issue 2052223:
         Taking the address of a protected function in a shared library
index cafe1a2df414e766d190ec434fd9dfa139f7db90..8dcc6d852a39f43dd07ab177cde4d221198e4201 100644 (file)
@@ -10041,6 +10041,7 @@ elf32_arm_final_link_relocate (reloc_howto_type *           howto,
     case R_ARM_ABS12:
       if (!globals->vxworks_p)
        return elf32_arm_abs12_reloc (input_bfd, hit_data, value + addend);
+      /* Fall through.  */
 
     case R_ARM_PC24:
     case R_ARM_ABS32:
index ecea3d79c12678737e2b71030192420df2cdba57..ef88f024f0f5af9298d7a248dbe4ce77ef95bc65 100644 (file)
@@ -2682,6 +2682,7 @@ elf32_avr_relax_section (bfd *abfd,
                   }
               }
           }
+         /* Fall through.  */
 
         default:
           {
index 6a3dabc0783a80e01271eed6fe3fad09c2cc016f..0a00803d3fe4cd7835daa4a2789b655b76209dcd 100644 (file)
@@ -2695,6 +2695,7 @@ bfinfdpic_relocate_section (bfd * output_bfd,
        case R_BFIN_BYTE4_DATA:
          if (! IS_FDPIC (output_bfd))
            goto non_fdpic;
+         /* Fall through.  */
 
        case R_BFIN_GOT17M4:
        case R_BFIN_GOTHI:
index 5d8d33045d676967bfdf0185d028c0ba377af57d..80f6959f81ad0985b629055ada50b36b67d70b84 100644 (file)
@@ -2775,6 +2775,7 @@ elf32_frv_relocate_section (bfd *output_bfd ATTRIBUTE_UNUSED,
        case R_FRV_32:
          if (! IS_FDPIC (output_bfd))
            goto non_fdpic;
+         /* Fall through.  */
 
        case R_FRV_GOT12:
        case R_FRV_GOTHI:
index 439e0d8738db9b0b636d66a8d987067ea43694ec..8d9638fa7ea96b72681712e278311d4624577eb8 100644 (file)
@@ -3951,7 +3951,7 @@ elf32_hppa_relocate_section (bfd *output_bfd,
                }
              plabel = 1;
            }
-         /* Fall through and possibly emit a dynamic relocation.  */
+         /* Fall through.  */
 
        case R_PARISC_DIR17F:
        case R_PARISC_DIR17R:
index 14a99ba5b926f3f6eff58573e8abcdfdba9651fb..c6f2fd6096fa6bcd6aa6e841413767dd06d91aba 100644 (file)
@@ -1947,6 +1947,7 @@ elf_i386_check_relocs (bfd *abfd,
 
            case R_386_GOTOFF:
              eh->gotoff_ref = 1;
+             /* Fall through.  */
            case R_386_32:
            case R_386_PC32:
            case R_386_PLT32:
index cc4b3ccff7e01492a86025d512780b7ddb57dd6d..778c6f048360a761e15644a94dff9a04d3663512 100644 (file)
@@ -1197,6 +1197,7 @@ microblaze_elf_relocate_section (bfd *output_bfd,
              goto dogot;
            case (int) R_MICROBLAZE_TLSLD:
              tls_type = (TLS_TLS | TLS_LD);
+             /* Fall through.  */
            dogot:
            case (int) R_MICROBLAZE_GOT_64:
              {
@@ -2401,8 +2402,10 @@ microblaze_elf_check_relocs (bfd * abfd,
           goto dogottls;
         case R_MICROBLAZE_TLSLD:
           tls_type |= (TLS_TLS | TLS_LD);
+         /* Fall through.  */
         dogottls:
           sec->has_tls_reloc = 1;
+         /* Fall through.  */
         case R_MICROBLAZE_GOT_64:
           if (htab->sgot == NULL)
             {
index 0f2d6dab6c2bdb1a53ce7da5d4649ca91d5b6a93..ac09e8742e39532248506b565ac641881cbcb848 100644 (file)
@@ -11229,6 +11229,7 @@ nds32_elf_relax_letlsls (struct bfd_link_info *link_info, bfd *abfd,
          success = 1;
          break;
        }
+      /* Fall through.  */
     case (N32_OP6_MEM << 8) | N32_MEM_LH:
     case (N32_OP6_MEM << 8) | N32_MEM_SH:
     case (N32_OP6_MEM << 8) | N32_MEM_LHS:
@@ -11243,6 +11244,7 @@ nds32_elf_relax_letlsls (struct bfd_link_info *link_info, bfd *abfd,
          success = 1;
          break;
        }
+      /* Fall through.  */
     case (N32_OP6_MEM << 8) | N32_MEM_LW:
     case (N32_OP6_MEM << 8) | N32_MEM_SW:
       /* The range is +/-64k.  */
@@ -11256,6 +11258,7 @@ nds32_elf_relax_letlsls (struct bfd_link_info *link_info, bfd *abfd,
          success = 1;
          break;
        }
+      /* Fall through.  */
     default:
       break;
     }
index f631b271f7e10ab848831b7d0f301c9256789725..627bff5b761fa9f7f534a868519bf4d8f3c07864 100644 (file)
@@ -4160,7 +4160,7 @@ ppc_elf_check_relocs (bfd *abfd,
          tls_type = TLS_TLS | TLS_DTPREL;
        dogottls:
          sec->has_tls_reloc = 1;
-         /* Fall thru */
+         /* Fall through.  */
 
          /* GOT16 relocations */
        case R_PPC_GOT16:
@@ -4232,7 +4232,7 @@ ppc_elf_check_relocs (bfd *abfd,
 
        case R_PPC_SDAREL16:
          htab->sdata[0].sym->ref_regular = 1;
-         /* Fall thru */
+         /* Fall through.  */
 
        case R_PPC_VLE_SDAREL_LO16A:
        case R_PPC_VLE_SDAREL_LO16D:
@@ -5167,7 +5167,7 @@ ppc_elf_gc_sweep_hook (bfd *abfd,
        case R_PPC_REL32:
          if (h == NULL || h == htab->elf.hgot)
            break;
-         /* Fall thru */
+         /* Fall through.  */
 
        case R_PPC_ADDR32:
        case R_PPC_ADDR24:
@@ -5182,6 +5182,7 @@ ppc_elf_gc_sweep_hook (bfd *abfd,
        case R_PPC_UADDR16:
          if (bfd_link_pic (info))
            break;
+         /* Fall through.  */
 
        case R_PPC_PLT32:
        case R_PPC_PLTREL24:
@@ -5409,7 +5410,7 @@ ppc_elf_tls_optimize (bfd *obfd ATTRIBUTE_UNUSED,
                    case R_PPC_GOT_TLSLD16:
                    case R_PPC_GOT_TLSLD16_LO:
                      expecting_tls_get_addr = 1;
-                     /* Fall thru */
+                     /* Fall through.  */
 
                    case R_PPC_GOT_TLSLD16_HI:
                    case R_PPC_GOT_TLSLD16_HA:
@@ -5427,7 +5428,7 @@ ppc_elf_tls_optimize (bfd *obfd ATTRIBUTE_UNUSED,
                    case R_PPC_GOT_TLSGD16:
                    case R_PPC_GOT_TLSGD16_LO:
                      expecting_tls_get_addr = 1;
-                     /* Fall thru */
+                     /* Fall through.  */
 
                    case R_PPC_GOT_TLSGD16_HI:
                    case R_PPC_GOT_TLSGD16_HA:
@@ -8154,7 +8155,7 @@ ppc_elf_relocate_section (bfd *output_bfd,
        case R_PPC_ADDR14_BRTAKEN:
        case R_PPC_REL14_BRTAKEN:
          branch_bit = BRANCH_PREDICT_BIT;
-         /* Fall thru */
+         /* Fall through.  */
 
          /* Branch not taken prediction relocations.  */
        case R_PPC_ADDR14_BRNTAKEN:
@@ -8978,7 +8979,7 @@ ppc_elf_relocate_section (bfd *output_bfd,
                              + htab->plt->output_offset
                              + ent->plt.offset);
            }
-         /* Fall thru */
+         /* Fall through.  */
 
        case R_PPC_RELAX:
          {
@@ -9480,7 +9481,7 @@ ppc_elf_relocate_section (bfd *output_bfd,
             alone (it will be set to zero elsewhere in the link).  */
          if (sec == NULL)
            break;
-         /* Fall thru */
+         /* Fall through.  */
 
        case R_PPC_PLT16_HA:
        case R_PPC_GOT16_HA:
index 3ba426f811e52bf56b9b2d60b0f67bc6ecbf9294..6cfed331cfb5703c0bc45ae2559a07025cee9124 100644 (file)
@@ -1958,6 +1958,7 @@ rl78_offset_for_reloc (bfd *                    abfd,
        default:
        reloc_computes_value:
          symval = rl78_compute_complex_reloc (r_type, symval, input_section);
+         /* Fall through.  */
        case R_RL78_DIR32:
        case R_RL78_DIR24S:
        case R_RL78_DIR16:
index 49556d2831ec79c709cebde23a9cf7370a62e4f6..7d5dc737bf2396dcccf9ca5ccf9f3d0387f72031 100644 (file)
@@ -704,6 +704,7 @@ rx_elf_relocate_section
        case R_RX_RH_8_NEG:
          WARN_REDHAT ("RX_RH_8_NEG");
          relocation = - relocation;
+         /* Fall through.  */
        case R_RX_DIR8S_PCREL:
          UNSAFE_FOR_PID;
          RANGE (-128, 127);
@@ -725,6 +726,7 @@ rx_elf_relocate_section
        case R_RX_RH_16_NEG:
          WARN_REDHAT ("RX_RH_16_NEG");
          relocation = - relocation;
+         /* Fall through.  */
        case R_RX_DIR16S_PCREL:
          UNSAFE_FOR_PID;
          RANGE (-32768, 32767);
@@ -809,6 +811,7 @@ rx_elf_relocate_section
          UNSAFE_FOR_PID;
          WARN_REDHAT ("RX_RH_24_NEG");
          relocation = - relocation;
+         /* Fall through.  */
        case R_RX_DIR24S_PCREL:
          RANGE (-0x800000, 0x7fffff);
 #if RX_OPCODE_BIG_ENDIAN
@@ -1232,6 +1235,7 @@ rx_elf_relocate_section
 
        case R_RX_ABS8S:
          UNSAFE_FOR_PID;
+         /* Fall through.  */
        case R_RX_ABS8S_PCREL:
          RX_STACK_POP (relocation);
          RANGE (-128, 127);
index 921268fa07582c4b807fe3d7988c36eaf4d0dd7e..7285ad6997a886468ff75684eb7f75a089df089c 100644 (file)
@@ -1124,6 +1124,7 @@ elf_s390_check_relocs (bfd *abfd,
        case R_390_GOTOFF32:
          if (h == NULL || !s390_is_ifunc_symbol_p (h) || !h->def_regular)
            break;
+         /* Fall through.  */
 
        case R_390_PLT12DBL:
        case R_390_PLT16DBL:
@@ -2802,6 +2803,7 @@ elf_s390_relocate_section (bfd *output_bfd,
                            + h ->plt.offset);
              goto do_relocation;
            }
+         /* Fall through.  */
 
        case R_390_8:
        case R_390_16:
index ec88a42e8c7645e2ad06aca99512e543a5bf10d0..9f84c4e8b03755db21c8c3b8e9987bf9b1b24549 100644 (file)
@@ -2746,7 +2746,7 @@ s3_bfd_score_elf_relocate_section (bfd *output_bfd,
 
             default:
               msg = _("internal error: unknown error");
-              /* fall through */
+              /* Fall through.  */
 
             common_error:
              (*info->callbacks->warning) (info, msg, name, input_bfd,
@@ -3655,7 +3655,8 @@ s3_bfd_score_elf_finish_dynamic_sections (bfd *output_bfd,
                 }
               /* In case if we don't have global got symbols we default
                   to setting DT_SCORE_GOTSYM to the same value as
-                  DT_SCORE_SYMTABNO, so we just fall through.  */
+                  DT_SCORE_SYMTABNO.  */
+             /* Fall through.  */
 
             case DT_SCORE_SYMTABNO:
               name = ".dynsym";
index 2c6da80e38ef2b0bf099c9b4057759c8c0f0ebf7..3aad1c7814830235ff45b3ccb6c94306158bff9e 100644 (file)
@@ -2551,7 +2551,7 @@ s7_bfd_score_elf_relocate_section (bfd *output_bfd,
 
             default:
               msg = _("internal error: unknown error");
-              /* fall through */
+              /* Fall through.  */
 
             common_error:
              (*info->callbacks->warning) (info, msg, name, input_bfd,
@@ -3466,7 +3466,8 @@ s7_bfd_score_elf_finish_dynamic_sections (bfd *output_bfd,
                 }
               /* In case if we don't have global got symbols we default
                   to setting DT_SCORE_GOTSYM to the same value as
-                  DT_SCORE_SYMTABNO, so we just fall through.  */
+                  DT_SCORE_SYMTABNO.  */
+             /* Fall through.  */
 
             case DT_SCORE_SYMTABNO:
               name = ".dynsym";
index d43ac54af39eb1caeb5150b9062fd55e87f90cd1..7f108fb173ae9fae98a314ab899dfd383b2bbe82 100644 (file)
@@ -5881,6 +5881,7 @@ sh_elf_check_relocs (bfd *abfd, struct bfd_link_info *info, asection *sec,
              /* This may require an rofixup.  */
              if (!htab->fdpic_p)
                break;
+             /* Fall through.  */
            case R_SH_GOTPLT32:
            case R_SH_GOT32:
            case R_SH_GOT20:
index c70b169ed8a1c6dbae65d3e4072b858b00e75731..d2dddbfbc8541a9f5363a73973c57c85eb1e994f 100644 (file)
@@ -2365,6 +2365,7 @@ elf32_tic6x_relocate_section (bfd *output_bfd,
                  goto done_reloc;
                }
            }
+         /* Fall through.  */
 
        case R_C6000_PCR_S12:
        case R_C6000_PCR_S10:
index 6f6d5e3c7bfd52cf26ab87b065b8dc8f99c7fd6a..a64e8db8fae2432fd23852d6e9bcac51e7452405 100644 (file)
@@ -3958,6 +3958,7 @@ elf64_ia64_relocate_section (bfd *output_bfd,
                       rel->r_offset, input_section->size);
                    break;
                  }
+               /* Fall through.  */
              default:
                (*info->callbacks->reloc_overflow) (info,
                                                    &h->root,
index 2a9ea1e2d261beea6a44fae81c4e284b7493a4d0..3264ecf4e68f95d09c850d606bc15b5513b7fed1 100644 (file)
@@ -5471,7 +5471,7 @@ ppc64_elf_check_relocs (bfd *abfd, struct bfd_link_info *info,
          tls_type = TLS_TLS | TLS_DTPREL;
        dogottls:
          sec->has_tls_reloc = 1;
-         /* Fall thru */
+         /* Fall through */
 
        case R_PPC64_GOT16:
        case R_PPC64_GOT16_DS:
@@ -5619,6 +5619,7 @@ ppc64_elf_check_relocs (bfd *abfd, struct bfd_link_info *info,
        case R_PPC64_TOC16_DS:
          htab->do_multi_toc = 1;
          ppc64_elf_tdata (abfd)->has_small_toc_reloc = 1;
+         /* Fall through.  */
        case R_PPC64_TOC16_LO:
        case R_PPC64_TOC16_HI:
        case R_PPC64_TOC16_HA:
@@ -8524,7 +8525,7 @@ ppc64_elf_tls_optimize (struct bfd_link_info *info)
                    case R_PPC64_GOT_TLSLD16_LO:
                      expecting_tls_get_addr = 1;
                      found_tls_get_addr_arg = 1;
-                     /* Fall thru */
+                     /* Fall through.  */
 
                    case R_PPC64_GOT_TLSLD16_HI:
                    case R_PPC64_GOT_TLSLD16_HA:
@@ -8544,7 +8545,7 @@ ppc64_elf_tls_optimize (struct bfd_link_info *info)
                    case R_PPC64_GOT_TLSGD16_LO:
                      expecting_tls_get_addr = 1;
                      found_tls_get_addr_arg = 1;
-                     /* Fall thru */
+                     /* Fall through. */
 
                    case R_PPC64_GOT_TLSGD16_HI:
                    case R_PPC64_GOT_TLSGD16_HA:
@@ -8575,7 +8576,7 @@ ppc64_elf_tls_optimize (struct bfd_link_info *info)
                    case R_PPC64_TLSGD:
                    case R_PPC64_TLSLD:
                      found_tls_get_addr_arg = 1;
-                     /* Fall thru */
+                     /* Fall through.  */
 
                    case R_PPC64_TLS:
                    case R_PPC64_TOC16:
@@ -9268,7 +9269,7 @@ ppc64_elf_edit_toc (struct bfd_link_info *info)
                            }
                          if ((opc & (0x3f << 2)) == (58u << 2))
                            break;
-                         /* Fall thru */
+                         /* Fall through.  */
 
                        default:
                          /* Wrong sort of reloc, or not a ld.  We may
@@ -14073,18 +14074,18 @@ ppc64_elf_relocate_section (bfd *output_bfd,
        case R_PPC64_ADDR14_BRTAKEN:
        case R_PPC64_REL14_BRTAKEN:
          insn = 0x01 << 21; /* 'y' or 't' bit, lowest bit of BO field.  */
-         /* Fall thru.  */
+         /* Fall through.  */
 
          /* Branch not taken prediction relocations.  */
        case R_PPC64_ADDR14_BRNTAKEN:
        case R_PPC64_REL14_BRNTAKEN:
          insn |= bfd_get_32 (output_bfd,
                              contents + rel->r_offset) & ~(0x01 << 21);
-         /* Fall thru.  */
+         /* Fall through.  */
 
        case R_PPC64_REL14:
          max_br_offset = 1 << 15;
-         /* Fall thru.  */
+         /* Fall through.  */
 
        case R_PPC64_REL24:
          /* Calls to functions with a different TOC, such as calls to
@@ -14730,7 +14731,7 @@ ppc64_elf_relocate_section (bfd *output_bfd,
        case R_PPC64_DTPREL64:
          if (htab->elf.tls_sec != NULL)
            addend -= htab->elf.tls_sec->vma + DTP_OFFSET;
-         /* Fall thru */
+         /* Fall through.  */
 
          /* Relocations that may need to be propagated if this is a
             dynamic object.  */
@@ -15073,7 +15074,7 @@ ppc64_elf_relocate_section (bfd *output_bfd,
             alone (it will be set to zero elsewhere in the link).  */
          if (sec == NULL)
            break;
-         /* Fall thru */
+         /* Fall through.  */
 
        case R_PPC64_GOT16_HA:
        case R_PPC64_PLTGOT16_HA:
index 431d06c271971d91596c9c2f5fca922238bd5eda..f9057e34b8623d424988fb6aa9f8feb3642fd707 100644 (file)
@@ -1052,6 +1052,7 @@ elf_s390_check_relocs (bfd *abfd,
        case R_390_GOTOFF64:
          if (h == NULL || !s390_is_ifunc_symbol_p (h) || !h->def_regular)
            break;
+         /* Fall through.  */
 
        case R_390_PLT12DBL:
        case R_390_PLT16DBL:
@@ -2758,6 +2759,7 @@ elf_s390_relocate_section (bfd *output_bfd,
                            + h->plt.offset);
              goto do_relocation;
            }
+         /* Fall through.  */
 
        case R_390_8:
        case R_390_16:
index 3cb5c8055dca9a6bafa155f29d81334a582afef1..2a372a6f3f5757bdd8844c9cbb8ce705cc8777ef 100644 (file)
@@ -2327,6 +2327,7 @@ elf_x86_64_check_relocs (bfd *abfd, struct bfd_link_info *info,
                        goto error_return;
                    }
                }
+             /* Fall through.  */
 
            case R_X86_64_32S:
            case R_X86_64_PC64:
@@ -2525,6 +2526,7 @@ elf_x86_64_check_relocs (bfd *abfd, struct bfd_link_info *info,
        case R_X86_64_32:
          if (!ABI_64_P (abfd))
            goto pointer;
+         /* Fall through.  */
        case R_X86_64_8:
        case R_X86_64_16:
        case R_X86_64_32S:
index d7207ff02e21847a01ff9254f59518f09d0f4cd6..1591682bce824fda63d2bd59dfff5b37d967455d 100644 (file)
@@ -7940,6 +7940,7 @@ eval_symbol (bfd_vma *result,
 
     case 'S':
       symbol_is_section = TRUE;
+      /* Fall through.  */
     case 's':
       ++sym;
       symlen = strtol (sym, (char **) symp, 10);
index b23db29f89e90ccc88635343cfa26a60bd4a12e9..56b0943037c97cd0673eff93ea63eff287c40df8 100644 (file)
@@ -5312,6 +5312,7 @@ elfNN_aarch64_final_link_relocate (reloc_howto_type *howto,
          bfd_set_error (bfd_error_bad_value);
          return FALSE;
        }
+      /* Fall through.  */
 
     case BFD_RELOC_AARCH64_16:
 #if ARCH_SIZE == 64
@@ -7323,6 +7324,7 @@ elfNN_aarch64_check_relocs (bfd *abfd, struct bfd_link_info *info,
              bfd_set_error (bfd_error_bad_value);
              return FALSE;
            }
+         /* Fall through.  */
 
        case BFD_RELOC_AARCH64_ADR_HI21_NC_PCREL:
        case BFD_RELOC_AARCH64_ADR_HI21_PCREL:
index 065409155af2f8429c9696677ad720df9300364f..a9a01fe599ad68ca71f8e94de652377a48bf0035 100644 (file)
@@ -4485,6 +4485,7 @@ missing_tls_sec:
                       rel->r_offset, input_section->size);
                    break;
                  }
+               /* Fall through.  */
              default:
                (*info->callbacks->reloc_overflow) (info,
                                                    &h->root,
index c8611702d870dcada7892e738187bb2f0e502e95..bdfb394b7324b0e87e602343013c636e72aeab21 100644 (file)
@@ -319,7 +319,7 @@ _bfd_aarch64_elf_put_addend (bfd *abfd,
          /* Force use of MOVZ.  */
          contents = reencode_movzn_to_movz (contents);
        }
-      /* fall through */
+      /* Fall through.  */
 
       /* Group relocations to create a 16, 32, 48 or 64 bit unsigned
          data or abs address inline.  */
index c1eeb717a3c8cae7031503ae9268e4a3e7a0b8f2..65f1ee4ea3aa03df44b6db60e19ea953cf5f3210 100644 (file)
@@ -11645,7 +11645,8 @@ _bfd_mips_elf_finish_dynamic_sections (bfd *output_bfd,
                }
              /* In case if we don't have global got symbols we default
                 to setting DT_MIPS_GOTSYM to the same value as
-                DT_MIPS_SYMTABNO, so we just fall through.  */
+                DT_MIPS_SYMTABNO.  */
+             /* Fall through.  */
 
            case DT_MIPS_SYMTABNO:
              name = ".dynsym";
index 2bcf58d3075a3f407e28e9c1504a2bf55f49441a..1893aee4f3cf46addaed90715c31c9176a61f350 100644 (file)
@@ -2597,10 +2597,13 @@ write_int (int value)
        {
        case 4:
          OUT (value >> 24);
+         /* Fall through.  */
        case 3:
          OUT (value >> 16);
+         /* Fall through.  */
        case 2:
          OUT (value >> 8);
+         /* Fall through.  */
        case 1:
          OUT (value);
        }
@@ -2753,15 +2756,19 @@ drop_int (struct output_buffer_struct *buf)
        case 0x84:
          ch = THIS ();
          NEXT ();
+         /* Fall through.  */
        case 0x83:
          ch = THIS ();
          NEXT ();
+         /* Fall through.  */
        case 0x82:
          ch = THIS ();
          NEXT ();
+         /* Fall through.  */
        case 0x81:
          ch = THIS ();
          NEXT ();
+         /* Fall through.  */
        case 0x80:
          break;
        }
@@ -2791,18 +2798,22 @@ copy_int (void)
          ch = THIS ();
          NEXT ();
          OUT (ch);
+         /* Fall through.  */
        case 0x83:
          ch = THIS ();
          NEXT ();
          OUT (ch);
+         /* Fall through.  */
        case 0x82:
          ch = THIS ();
          NEXT ();
          OUT (ch);
+         /* Fall through.  */
        case 0x81:
          ch = THIS ();
          NEXT ();
          OUT (ch);
+         /* Fall through.  */
        case 0x80:
          break;
        }
@@ -2833,12 +2844,15 @@ copy_till_end (void)
        case 0x84:
          OUT (THIS ());
          NEXT ();
+         /* Fall through.  */
        case 0x83:
          OUT (THIS ());
          NEXT ();
+         /* Fall through.  */
        case 0x82:
          OUT (THIS ());
          NEXT ();
+         /* Fall through.  */
        case 0x81:
          OUT (THIS ());
          NEXT ();
index cbc35788f369edf787d2c394f6c1f49d81440619..5560d38116d76a32c965fceb37915e69fe5f0b43 100644 (file)
@@ -612,6 +612,7 @@ oasys_object_p (bfd *abfd)
          break;
        case oasys_record_is_data_enum:
          oasys->first_data_record = bfd_tell (abfd) - record.header.length;
+         /* Fall through.  */
        case oasys_record_is_debug_enum:
        case oasys_record_is_module_enum:
        case oasys_record_is_named_section_enum:
index 7daf52d38204b34ce1f7845f1aa22333bcb03440..089c917bff95ec77a6e87e450e5bbb2b273a19e8 100644 (file)
@@ -2985,6 +2985,7 @@ aout_link_write_other_symbol (struct bfd_hash_entry *bh, void *data)
     case bfd_link_hash_undefweak:
       type = N_WEAKU;
       val = 0;
+      /* Fall through.  */
     case bfd_link_hash_indirect:
     case bfd_link_hash_warning:
       /* FIXME: Ignore these for now.  The circumstances under which
index a013f9bb8ccc79c923d89c1ba38a3cca2dc0ee02..3197ec05b24b2dacc9d1cbc303962dc374d5efed 100644 (file)
@@ -972,10 +972,12 @@ srec_write_record (bfd *abfd,
     case 7:
       TOHEX (dst, (address >> 24), check_sum);
       dst += 2;
+      /* Fall through.  */
     case 8:
     case 2:
       TOHEX (dst, (address >> 16), check_sum);
       dst += 2;
+      /* Fall through.  */
     case 9:
     case 1:
     case 0:
index ed46e3baf43952b9e3151f88497346ab7bac9966..cef4b68466286bc0c6220ae5e0bb0d6814a05724 100644 (file)
@@ -296,6 +296,7 @@ process_esd (bfd *abfd, struct ext_esd *esd, int pass)
          break;
        case ESD_XDEF_IN_ABS:
          sec = bfd_abs_section_ptr;
+         /* Fall through.  */
        case ESD_XDEF_IN_SEC:
          {
            int snum = VDATA (abfd)->def_idx++;
index 4380b6381dd2611e2922df0de6051131cca52480..8512827b4ed23fe25eacc4f83767116c6a567059 100644 (file)
@@ -1,3 +1,13 @@
+2016-10-06  Alan Modra  <amodra@gmail.com>
+
+       * dlltool.c: Spell fall through comments consistently.
+       * objcopy.c: Likewise.
+       * readelf.c: Likewise.
+       * dwarf.c: Add missing fall through comments.
+       * elfcomm.c: Likewise.
+       * sysinfo.y: Likewise.
+       * readelf.c: Likewise.  Also remove extraneous comments.
+
 2016-10-06  Alan Modra  <amodra@gmail.com>
 
        * cxxfilt.c (usage): Add ATTRIBUTE_NORETURN.
index e116e6bf9c2e3beaee9b0de3c1be72365112af55..e7b40eee46d49d6274ae6329967c5b4334ed2349 100644 (file)
@@ -2708,7 +2708,8 @@ make_one_lib_file (export_type *exp, int i, int delay)
              sec->orelocation = rpp;
              break;
            }
-         /* else fall through */
+         /* Fall through.  */
+
        case IDATA4:
          /* An idata$4 or idata$5 is one word long, and has an
             rva to idata$6.  */
index b4687e97c9022e735d1bee93c3fa5f7aeea05848..fee6a605e71d452fcb3166b7dac166b548b0b65b 100644 (file)
@@ -1824,6 +1824,7 @@ read_and_display_attr_value (unsigned long attribute,
        {
        case DW_AT_frame_base:
          have_frame_base = 1;
+         /* Fall through.  */
        case DW_AT_location:
        case DW_AT_string_length:
        case DW_AT_return_addr:
@@ -2099,6 +2100,7 @@ read_and_display_attr_value (unsigned long attribute,
 
     case DW_AT_frame_base:
       have_frame_base = 1;
+      /* Fall through.  */
     case DW_AT_location:
     case DW_AT_string_length:
     case DW_AT_return_addr:
@@ -7654,6 +7656,7 @@ dwarf_select_sections_by_letters (const char *letters)
 
       case 'F':
        do_debug_frames_interp = 1;
+       /* Fall through.  */
       case 'f':
        do_debug_frames = 1;
        break;
index b17627a2b009d2a86c5d89db180ba9fc72ba1ce7..1b3218649f4f798c4107b5e031438a84eb52be74 100644 (file)
@@ -165,6 +165,7 @@ byte_get_little_endian (unsigned char *field, int size)
          |    (((unsigned long) (field[1])) << 8)
          |    (((unsigned long) (field[2])) << 16)
          |    (((unsigned long) (field[3])) << 24);
+      /* Fall through.  */
 
     case 6:
       if (sizeof (elf_vma) == 8)
@@ -182,6 +183,7 @@ byte_get_little_endian (unsigned char *field, int size)
          |    (((unsigned long) (field[1])) << 8)
          |    (((unsigned long) (field[2])) << 16)
          |    (((unsigned long) (field[3])) << 24);
+      /* Fall through.  */
 
     case 7:
       if (sizeof (elf_vma) == 8)
@@ -200,6 +202,7 @@ byte_get_little_endian (unsigned char *field, int size)
          |    (((unsigned long) (field[1])) << 8)
          |    (((unsigned long) (field[2])) << 16)
          |    (((unsigned long) (field[3])) << 24);
+      /* Fall through.  */
 
     case 8:
       if (sizeof (elf_vma) == 8)
@@ -219,6 +222,7 @@ byte_get_little_endian (unsigned char *field, int size)
          |    (((unsigned long) (field[1])) << 8)
          |    (((unsigned long) (field[2])) << 16)
          |    (((unsigned long) (field[3])) << 24);
+      /* Fall through.  */
 
     default:
       error (_("Unhandled data length: %d\n"), size);
@@ -265,6 +269,7 @@ byte_get_big_endian (unsigned char *field, int size)
            |   (((unsigned long) (field[1])) << 16)
            |   (((unsigned long) (field[0])) << 24);
        }
+      /* Fall through.  */
 
     case 6:
       if (sizeof (elf_vma) == 8)
@@ -284,6 +289,7 @@ byte_get_big_endian (unsigned char *field, int size)
            |   (((unsigned long) (field[1])) << 16)
            |   (((unsigned long) (field[0])) << 24);
        }
+      /* Fall through.  */
 
     case 7:
       if (sizeof (elf_vma) == 8)
@@ -304,6 +310,7 @@ byte_get_big_endian (unsigned char *field, int size)
            |   (((unsigned long) (field[1])) << 16)
            |   (((unsigned long) (field[0])) << 24);
        }
+      /* Fall through.  */
 
     case 8:
       if (sizeof (elf_vma) == 8)
@@ -325,6 +332,7 @@ byte_get_big_endian (unsigned char *field, int size)
            |   (((unsigned long) (field[1])) << 16)
            |   (((unsigned long) (field[0])) << 24);
        }
+      /* Fall through.  */
 
     default:
       error (_("Unhandled data length: %d\n"), size);
index 9a64ec30e652f1e9370f1a271b612010287f2495..1821c0ce1be5c1cd39106b9b9fe67fa22c8a6c21 100644 (file)
@@ -4302,7 +4302,7 @@ copy_main (int argc, char *argv[])
              {
              case OPTION_CHANGE_SECTION_ADDRESS:
                p->vma_val = val;
-               /* Drop through.  */
+               /* Fall through.  */
 
              case OPTION_CHANGE_SECTION_LMA:
                p->lma_val = val;
index 9a07158cc398fa77aee764780a5c9213f8cd85ab..8ca8f4b33b1a7a7a155c83af92d080dc8de95d6a 100644 (file)
@@ -426,7 +426,7 @@ print_vma (bfd_vma vma, print_mode mode)
     {
     case FULL_HEX:
       nc = printf ("0x");
-      /* Drop through.  */
+      /* Fall through.  */
 
     case LONG_HEX:
 #ifdef BFD64
@@ -439,11 +439,11 @@ print_vma (bfd_vma vma, print_mode mode)
     case DEC_5:
       if (vma <= 99999)
        return printf ("%5" BFD_VMA_FMT "d", vma);
-      /* Drop through.  */
+      /* Fall through.  */
 
     case PREFIX_HEX:
       nc = printf ("0x");
-      /* Drop through.  */
+      /* Fall through.  */
 
     case HEX:
       return nc + printf ("%" BFD_VMA_FMT "x", vma);
@@ -1292,6 +1292,7 @@ dump_relocations (FILE * file,
              rtype = elf_msp430x_reloc_type (type);
              break;
            }
+         /* Fall through.  */
        case EM_MSP430_OLD:
          rtype = elf_msp430_reloc_type (type);
          break;
@@ -4450,7 +4451,7 @@ parse_args (int argc, char ** argv)
        default:
          /* xgettext:c-format */
          error (_("Invalid option '-%c'\n"), c);
-         /* Drop through.  */
+         /* Fall through.  */
        case '?':
          usage (stderr);
        }
@@ -9679,6 +9680,7 @@ process_dynamic_section (FILE * file)
        case DT_SYMENT  :
        case DT_RELENT  :
          dynamic_info[entry->d_tag] = entry->d_un.d_val;
+         /* Fall through.  */
        case DT_PLTPADSZ:
        case DT_MOVEENT :
        case DT_MOVESZ  :
@@ -11494,6 +11496,7 @@ target_specific_reloc_handling (Elf_Internal_Rela * reloc,
          case 10: /* R_MSP430_SYM_DIFF */
            if (uses_msp430x_relocs ())
              break;
+           /* Fall through.  */
          case 21: /* R_MSP430X_SYM_DIFF */
            saved_sym = symtab + get_reloc_symindex (reloc->r_info);
            return TRUE;
@@ -12011,6 +12014,7 @@ is_16bit_abs_reloc (unsigned int reloc_type)
     case EM_MSP430:
       if (uses_msp430x_relocs ())
        return reloc_type == 2; /* R_MSP430_ABS16.  */
+      /* Fall through.  */
     case EM_MSP430_OLD:
       return reloc_type == 5; /* R_MSP430_16_BYTE.  */
     case EM_NDS32:
@@ -14241,6 +14245,7 @@ process_attributes (FILE * file,
                      goto do_numlist;
                    case 3:
                      printf (_("Symbol Attributes:"));
+                     /* Fall through.  */
                    do_numlist:
                      for (;;)
                        {
@@ -16418,8 +16423,8 @@ get_file_header (FILE * file)
   /* Determine how to read the rest of the header.  */
   switch (elf_header.e_ident[EI_DATA])
     {
-    default: /* fall through */
-    case ELFDATANONE: /* fall through */
+    default:
+    case ELFDATANONE:
     case ELFDATA2LSB:
       byte_get = byte_get_little_endian;
       byte_put = byte_put_little_endian;
index 9df3cb324c53922ec901e8f135d7c7b215dd9c28..10fb49a8450083519d94dc73d1483666d760e68e 100644 (file)
@@ -153,9 +153,8 @@ it:
     break;
   case 'g':
     printf("\tchecksum(ffile,raw, idx, IT_%s_CODE);\n", it);
-
+    /* Fall through.  */
   case 'i':
-
   case 'o':
   case 'c':
     printf("}\n");
@@ -182,6 +181,7 @@ repeat_it_field: '(' REPEAT NAME
              printf("\tprintf(\"repeat %%d\\n\", %s);\n",$3);
              if (rdepth==2)
              printf("\tprintf(\"repeat %%d\\n\", %s[n]);\n",$3);
+             /* Fall through.  */
            case 'i':
            case 'g':
            case 'o':
index 5ccc133e92b7072a2dbc1fa1c6c366d91894fc83..d0b848934f3f08baf5a23764df56306d7f0c08ca 100644 (file)
@@ -1,3 +1,51 @@
+2016-10-06  Alan Modra  <amodra@gmail.com>
+
+       * app.c: Add missing fall through comments.
+       * dw2gencfi.c: Likewise.
+       * expr.c: Likewise.
+       * config/tc-alpha.c: Likewise.
+       * config/tc-arc.c: Likewise.
+       * config/tc-arm.c: Likewise.
+       * config/tc-cr16.c: Likewise.
+       * config/tc-crx.c: Likewise.
+       * config/tc-dlx.c: Likewise.
+       * config/tc-h8300.c: Likewise.
+       * config/tc-hppa.c: Likewise.
+       * config/tc-i370.c: Likewise.
+       * config/tc-i386.c: Likewise.
+       * config/tc-i960.c: Likewise.
+       * config/tc-ia64.c: Likewise.
+       * config/tc-m68hc11.c: Likewise.
+       * config/tc-m68k.c: Likewise.
+       * config/tc-mep.c: Likewise.
+       * config/tc-metag.c: Likewise.
+       * config/tc-microblaze.c: Likewise.
+       * config/tc-mips.c: Likewise.
+       * config/tc-ns32k.c: Likewise.
+       * config/tc-rx.c: Likewise.
+       * config/tc-score.c: Likewise.
+       * config/tc-score7.c: Likewise.
+       * config/tc-sh.c: Likewise.
+       * config/tc-tic4x.c: Likewise.
+       * config/tc-vax.c: Likewise.
+       * config/tc-xstormy16.c: Likewise.
+       * config/tc-z80.c: Likewise.
+       * config/tc-z8k.c: Likewise.
+       * config/obj-elf.c: Likewise.
+       * config/tc-i386.c: Likewise.
+       * depend.c: Spell fall through comments consistently.
+       * config/tc-arm.c: Likewise.
+       * config/tc-d10v.c: Likewise.
+       * config/tc-i960.c: Likewise.
+       * config/tc-ia64.c: Likewise.
+       * config/tc-m68k.c: Likewise.
+       * config/tc-mcore.c: Likewise.
+       * config/tc-mep.c: Likewise.
+       * config/tc-ns32k.c: Likewise.
+       * config/tc-visium.c: Likewise.
+       * config/tc-xstormy16.c: Likewise.
+       * config/tc-z8k.c: Likewise.
+
 2016-10-06  Alan Modra  <amodra@gmail.com>
 
        * as.h (as_assert): Add ATTRIBUTE_NORETURN.
index 38d8e78f211f9deefe28ccd3e3fbc14cc2a82c38..4b53ce706127b55f56b0980d6ec42671202b015b 100644 (file)
--- a/gas/app.c
+++ b/gas/app.c
@@ -695,6 +695,7 @@ do_scrub_chars (size_t (*get) (char *, size_t), char *tostart, size_t tolen)
              state = 9;
              break;
            }
+         /* Fall through.  */
        case 17:
          /* We have seen "af" at the start of a symbol,
             a ' here is a part of that symbol.  */
index 8af563fbd24e6d1c6c1b99175d77248eace9745a..8d80c77f1535e03a39602702df6f80d7cfee6a97 100644 (file)
@@ -813,6 +813,7 @@ obj_elf_parse_section_letters (char *str, size_t len, bfd_boolean *is_clone)
                }
              break;
            }
+         /* Fall through.  */
        default:
          {
            const char *bad_msg = _("unrecognized .section attribute:"
index c70b1648b2b85136ae89c2dd6a643797e19566f6..f7656ec7ee253bb5ee71eb323b43cac0065b0f9c 100644 (file)
@@ -1005,6 +1005,7 @@ tokenize_arguments (char *str,
            /* ... then fall through to plain expression.  */
            input_line_pointer = hold;
          }
+         /* Fall through.  */
 
        default:
          if (saw_arg && !saw_comma)
@@ -5583,6 +5584,7 @@ md_atof (int type, char *litP, int *sizeP)
     case 'G':
       /* vax_md_atof() doesn't like "G" for some reason.  */
       type = 'g';
+      /* Fall through.  */
     case 'F':
     case 'D':
       return vax_md_atof (type, litP, sizeP);
index d480bf1a1e9f8b7fc331629ed82ba10d71d47023..f28abf492a6116aeb042c9fb87dbaf7a41226b1b 100644 (file)
@@ -1836,7 +1836,7 @@ find_opcode_match (const struct arc_opcode_hash_entry *entry,
                    if (tok[tokidx].X_op != O_constant)
                      goto de_fault;
                  }
-                 /* Fall-through */
+                 /* Fall through.  */
                case O_constant:
                  /* Check the range.  */
                  if (operand->bits != 32
@@ -1908,6 +1908,7 @@ find_opcode_match (const struct arc_opcode_hash_entry *entry,
                        goto match_failed;
                      break;
                    }
+                 /* Fall through.  */
                default:
                de_fault:
                  if (operand->default_reloc == 0)
@@ -1926,6 +1927,7 @@ find_opcode_match (const struct arc_opcode_hash_entry *entry,
                    case O_tlsie:
                      if (!(operand->flags & ARC_OPERAND_LIMM))
                        goto match_failed;
+                     /* Fall through.  */
                    case O_absent:
                      if (!generic_reloc_p (operand->default_reloc))
                        goto match_failed;
@@ -3022,6 +3024,7 @@ md_apply_fix (fixS *fixP,
     case BFD_RELOC_ARC_TLS_LE_32:
       gas_assert (!fixP->fx_addsy);
       gas_assert (!fixP->fx_subsy);
+      /* Fall through.  */
 
     case BFD_RELOC_ARC_GOTOFF:
     case BFD_RELOC_ARC_32_ME:
@@ -3047,6 +3050,7 @@ md_apply_fix (fixS *fixP,
 
     case BFD_RELOC_ARC_S21W_PCREL_PLT:
       reloc = BFD_RELOC_ARC_S21W_PCREL;
+      /* Fall through.  */
 
     case BFD_RELOC_ARC_S25W_PCREL:
     case BFD_RELOC_ARC_S21W_PCREL:
@@ -3868,6 +3872,7 @@ assemble_insn (const struct arc_opcode *opcode,
              image = insert_operand (image, operand, regs, NULL, 0);
              break;
            }
+         /* Fall through.  */
 
        default:
          /* This operand needs a relocation.  */
index db6dd681358e7e12ee0ba45314e9745a887d5ef7..526131c5eae1ac8705b47f42e98badc96b9c9f48 100644 (file)
@@ -1276,6 +1276,7 @@ arm_reg_alt_syntax (char **ccp, char *start, struct reg_entry *reg,
        if (*ccp != start && processor <= 15)
          return processor;
       }
+      /* Fall through.  */
 
     case REG_TYPE_MMXWC:
       /* WC includes WCG.  ??? I'm not sure this is true for all
@@ -17773,7 +17774,7 @@ opcode_lookup (char **str)
        case OT_odd_infix_unc:
          if (!unified_syntax)
            return 0;
-         /* else fall through */
+         /* Fall through.  */
 
        case OT_csuffix:
        case OT_csuffixF:
@@ -22842,6 +22843,7 @@ md_apply_fix (fixS *    fixP,
     case BFD_RELOC_ARM_OFFSET_IMM:
       if (!fixP->fx_done && seg->use_rela_p)
        value = 0;
+      /* Fall through.  */
 
     case BFD_RELOC_ARM_LITERAL:
       sign = value > 0;
@@ -23200,6 +23202,7 @@ md_apply_fix (fixS *    fixP,
          newval = md_chars_to_number (buf, INSN_SIZE);
          fixP->fx_done = 0;
        }
+      /* Fall through.  */
 
     case BFD_RELOC_ARM_PLT32:
 #endif
@@ -24092,6 +24095,7 @@ tc_gen_reloc (asection *section, fixS *fixp)
          code = BFD_RELOC_8_PCREL;
          break;
        }
+      /* Fall through.  */
 
     case BFD_RELOC_16:
       if (fixp->fx_pcrel)
@@ -24099,6 +24103,7 @@ tc_gen_reloc (asection *section, fixS *fixp)
          code = BFD_RELOC_16_PCREL;
          break;
        }
+      /* Fall through.  */
 
     case BFD_RELOC_32:
       if (fixp->fx_pcrel)
@@ -24106,6 +24111,7 @@ tc_gen_reloc (asection *section, fixS *fixp)
          code = BFD_RELOC_32_PCREL;
          break;
        }
+      /* Fall through.  */
 
     case BFD_RELOC_ARM_MOVW:
       if (fixp->fx_pcrel)
@@ -24113,6 +24119,7 @@ tc_gen_reloc (asection *section, fixS *fixp)
          code = BFD_RELOC_ARM_MOVW_PCREL;
          break;
        }
+      /* Fall through.  */
 
     case BFD_RELOC_ARM_MOVT:
       if (fixp->fx_pcrel)
@@ -24120,6 +24127,7 @@ tc_gen_reloc (asection *section, fixS *fixp)
          code = BFD_RELOC_ARM_MOVT_PCREL;
          break;
        }
+      /* Fall through.  */
 
     case BFD_RELOC_ARM_THUMB_MOVW:
       if (fixp->fx_pcrel)
@@ -24127,6 +24135,7 @@ tc_gen_reloc (asection *section, fixS *fixp)
          code = BFD_RELOC_ARM_THUMB_MOVW_PCREL;
          break;
        }
+      /* Fall through.  */
 
     case BFD_RELOC_ARM_THUMB_MOVT:
       if (fixp->fx_pcrel)
@@ -24134,6 +24143,7 @@ tc_gen_reloc (asection *section, fixS *fixp)
          code = BFD_RELOC_ARM_THUMB_MOVT_PCREL;
          break;
        }
+      /* Fall through.  */
 
     case BFD_RELOC_NONE:
     case BFD_RELOC_ARM_PCREL_BRANCH:
index 40d9ba4d510f193c9dd543ab2134575c27fea310..186cfb9a56cffb6da0317a6c3cd8f0bbb54b862f 100644 (file)
@@ -1165,6 +1165,7 @@ set_operand (char *operand, ins * cr16_ins)
     {
     case arg_ic:    /* Case $0x18.  */
       operandS++;
+      /* Fall through.  */
     case arg_c:     /* Case 0x18.  */
       /* Set constant.  */
       process_label_constant (operandS, cr16_ins);
@@ -1182,6 +1183,7 @@ set_operand (char *operand, ins * cr16_ins)
       *operandE = '\0';
       process_label_constant (operandS, cr16_ins);
       operandS = operandE;
+      /* Fall through.  */
     case arg_rbase: /* Case (r1) or (r1,r0).  */
       operandS++;
       /* Set register base.  */
@@ -1789,7 +1791,9 @@ print_constant (int nbits, int shift, argument *arg)
       break;
 
     case 21:
-      if ((nbits == 21) && (IS_INSN_TYPE (LD_STOR_INS))) nbits = 20;
+      if ((nbits == 21) && (IS_INSN_TYPE (LD_STOR_INS)))
+       nbits = 20;
+      /* Fall through.  */
     case 24:
     case 22:
     case 20:
index 9a943d530feaf45d03c3aa61e2d309ada3b2801b..a9af5bb83adc0d59b3aea272f2a6343b0072d87d 100644 (file)
@@ -730,6 +730,7 @@ set_operand (char *operand, ins * crx_ins)
     case arg_sc:    /* Case *+0x18.  */
     case arg_ic:    /* Case $0x18.  */
       operandS++;
+      /* Fall through.  */
     case arg_c:            /* Case 0x18.  */
       /* Set constant.  */
       process_label_constant (operandS, crx_ins);
@@ -747,6 +748,7 @@ set_operand (char *operand, ins * crx_ins)
       *operandE = '\0';
       process_label_constant (operandS, crx_ins);
       operandS = operandE;
+      /* Fall through.  */
     case arg_rbase: /* Case (r1).  */
       operandS++;
       /* Set register base.  */
@@ -1279,6 +1281,7 @@ print_operand (int nbits, int shift, argument *arg)
       CRX_PRINT (0, getreg_image (arg->r), 12);
       CRX_PRINT (0, getreg_image (arg->i_r), 8);
       CRX_PRINT (0, arg->scale, 6);
+      /* Fall through.  */
     case arg_ic:
     case arg_c:
       print_constant (nbits, shift, arg);
index 03b264d63cf25d7e926c4a56d2a459f37fe054aa..e7cfcdfa9a1ff8ad6889d07ac878a0fd1ae31306 100644 (file)
@@ -1548,7 +1548,7 @@ md_apply_fix (fixS *fixP, valueT *valP, segT seg ATTRIBUTE_UNUSED)
          if ( segf && segf->sym != fixP->fx_addsy)
            value = 0;
         }
-      /* Drop through.  */
+      /* Fall through.  */
     case BFD_RELOC_D10V_18:
       /* Instruction addresses are always right-shifted by 2.  */
       value >>= AT_WORD_RIGHT_SHIFT;
index 079a9b6ae97c06725ae5bee7fe9f60c3af533730..a03aa196fb64547f9d375d2a053b489d29d90e98 100644 (file)
@@ -780,6 +780,7 @@ machine_ip (char *str)
              reg_shift = 21;
              goto general_reg;
            }
+         /* Fall through.  */
 
          /* The immediate 16 bits literal, bit 0-15.  */
        case 'i':
index 048d31ea3c3e25d0f886c74b1c5c4acb3083c0aa..28566a54c6509687afb833e0fd6f5d7e375aacd5 100644 (file)
@@ -1432,6 +1432,7 @@ do_a_fix_imm (int offset, int nibble, struct h8_op *operand, int relaxmode, cons
          break;
        default:
          as_bad (_("Can't work out size of operand.\n"));
+         /* Fall through.  */
        case L_16:
        case L_16U:
          size = 2;
index f5cdc2c09ff9ae23cddef28d551c719a96dc788f..2ed06a22233a1757962ed4e398dac420222f493a 100644 (file)
@@ -5432,6 +5432,7 @@ pa_ip (char *str)
                {
                case SGL:
                  opcode |= 0x20;
+                 /* Fall through.  */
                case DBL:
                  the_insn.fpof1 = flag;
                  continue;
index ac8aa95c0e4dd991833702951a118845785b8d9a..6299cc7610aac94782b35a2b5f24530186c01bdc 100644 (file)
@@ -956,6 +956,7 @@ i370_dc (int unused ATTRIBUTE_UNUSED)
       break;
     case 'E':  /* 32-bit */
       type = 'f';
+      /* Fall through.  */
     case 'D':  /* 64-bit */
       md_atof (type, tmp, &nbytes);
       p = frag_more (nbytes);
index 660d23e68f28b17e55cc12cd5756ca4544cc35bc..0ab9e1e34bbd5e2355b6cc1a852272448fab2569 100644 (file)
@@ -1374,9 +1374,11 @@ operand_type_all_zero (const union i386_operand_type *x)
     case 3:
       if (x->array[2])
        return 0;
+      /* Fall through.  */
     case 2:
       if (x->array[1])
        return 0;
+      /* Fall through.  */
     case 1:
       return !x->array[0];
     default:
@@ -1391,10 +1393,13 @@ operand_type_set (union i386_operand_type *x, unsigned int v)
     {
     case 3:
       x->array[2] = v;
+      /* Fall through.  */
     case 2:
       x->array[1] = v;
+      /* Fall through.  */
     case 1:
       x->array[0] = v;
+      /* Fall through.  */
       break;
     default:
       abort ();
@@ -1410,9 +1415,11 @@ operand_type_equal (const union i386_operand_type *x,
     case 3:
       if (x->array[2] != y->array[2])
        return 0;
+      /* Fall through.  */
     case 2:
       if (x->array[1] != y->array[1])
        return 0;
+      /* Fall through.  */
     case 1:
       return x->array[0] == y->array[0];
       break;
@@ -1429,9 +1436,11 @@ cpu_flags_all_zero (const union i386_cpu_flags *x)
     case 3:
       if (x->array[2])
        return 0;
+      /* Fall through.  */
     case 2:
       if (x->array[1])
        return 0;
+      /* Fall through.  */
     case 1:
       return !x->array[0];
     default:
@@ -1448,9 +1457,11 @@ cpu_flags_equal (const union i386_cpu_flags *x,
     case 3:
       if (x->array[2] != y->array[2])
        return 0;
+      /* Fall through.  */
     case 2:
       if (x->array[1] != y->array[1])
        return 0;
+      /* Fall through.  */
     case 1:
       return x->array[0] == y->array[0];
       break;
@@ -1473,8 +1484,10 @@ cpu_flags_and (i386_cpu_flags x, i386_cpu_flags y)
     {
     case 3:
       x.array [2] &= y.array [2];
+      /* Fall through.  */
     case 2:
       x.array [1] &= y.array [1];
+      /* Fall through.  */
     case 1:
       x.array [0] &= y.array [0];
       break;
@@ -1491,8 +1504,10 @@ cpu_flags_or (i386_cpu_flags x, i386_cpu_flags y)
     {
     case 3:
       x.array [2] |= y.array [2];
+      /* Fall through.  */
     case 2:
       x.array [1] |= y.array [1];
+      /* Fall through.  */
     case 1:
       x.array [0] |= y.array [0];
       break;
@@ -1509,8 +1524,10 @@ cpu_flags_and_not (i386_cpu_flags x, i386_cpu_flags y)
     {
     case 3:
       x.array [2] &= ~y.array [2];
+      /* Fall through.  */
     case 2:
       x.array [1] &= ~y.array [1];
+      /* Fall through.  */
     case 1:
       x.array [0] &= ~y.array [0];
       break;
@@ -1607,8 +1624,10 @@ operand_type_and (i386_operand_type x, i386_operand_type y)
     {
     case 3:
       x.array [2] &= y.array [2];
+      /* Fall through.  */
     case 2:
       x.array [1] &= y.array [1];
+      /* Fall through.  */
     case 1:
       x.array [0] &= y.array [0];
       break;
@@ -1625,8 +1644,10 @@ operand_type_or (i386_operand_type x, i386_operand_type y)
     {
     case 3:
       x.array [2] |= y.array [2];
+      /* Fall through.  */
     case 2:
       x.array [1] |= y.array [1];
+      /* Fall through.  */
     case 1:
       x.array [0] |= y.array [0];
       break;
@@ -1643,8 +1664,10 @@ operand_type_xor (i386_operand_type x, i386_operand_type y)
     {
     case 3:
       x.array [2] ^= y.array [2];
+      /* Fall through.  */
     case 2:
       x.array [1] ^= y.array [1];
+      /* Fall through.  */
     case 1:
       x.array [0] ^= y.array [0];
       break;
@@ -3928,6 +3951,7 @@ check_suffix:
          if (intel_syntax && (intel_float_operand (mnemonic) & 2))
            i.suffix = SHORT_MNEM_SUFFIX;
          else
+           /* Fall through.  */
        case BYTE_MNEM_SUFFIX:
        case QWORD_MNEM_SUFFIX:
          i.suffix = mnem_p[-1];
@@ -4203,6 +4227,7 @@ swap_operands (void)
     case 5:
     case 4:
       swap_2_operands (1, i.operands - 2);
+      /* Fall through.  */
     case 3:
     case 2:
       swap_2_operands (0, i.operands - 1);
@@ -4942,11 +4967,13 @@ match_template (char mnem_suffix)
              else if (t->opcode_modifier.d)
                goto check_reverse;
            }
+         /* Fall through.  */
 
        case 3:
          /* If we swap operand in encoding, we match the next one.  */
          if (i.swap_operand && t->opcode_modifier.s)
            continue;
+         /* Fall through.  */
        case 4:
        case 5:
          overlap1 = operand_type_and (i.types[1], operand_types[1]);
@@ -4998,9 +5025,11 @@ check_reverse:
                case 5:
                  overlap4 = operand_type_and (i.types[4],
                                               operand_types[4]);
+                 /* Fall through.  */
                case 4:
                  overlap3 = operand_type_and (i.types[3],
                                               operand_types[3]);
+                 /* Fall through.  */
                case 3:
                  overlap2 = operand_type_and (i.types[2],
                                               operand_types[2]);
@@ -5018,6 +5047,7 @@ check_reverse:
                                                       i.types[4],
                                                       operand_types[4]))
                    continue;
+                 /* Fall through.  */
                case 4:
                  if (!operand_type_match (overlap3, i.types[3])
                      || (check_register
@@ -5028,6 +5058,7 @@ check_reverse:
                                                           i.types[3],
                                                           operand_types[3])))
                    continue;
+                 /* Fall through.  */
                case 3:
                  /* Here we make use of the fact that there are no
                     reverse match 3 operand instructions, and all 3
@@ -5370,6 +5401,7 @@ process_suffix (void)
              i.suffix = QWORD_MNEM_SUFFIX;
              break;
            }
+         /* Fall through.  */
        case CODE_32BIT:
          if (!i.tm.opcode_modifier.no_lsuf)
            i.suffix = LONG_MNEM_SUFFIX;
@@ -6953,6 +6985,7 @@ output_jump (void)
     {
     case 2:
       *p++ = i.tm.base_opcode >> 8;
+      /* Fall through.  */
     case 1:
       *p++ = i.tm.base_opcode;
       break;
@@ -10694,6 +10727,7 @@ tc_gen_reloc (asection *section ATTRIBUTE_UNUSED, fixS *fixp)
          return NULL;
        }
 #endif
+      /* Fall through.  */
 
     case BFD_RELOC_X86_64_PLT32:
     case BFD_RELOC_X86_64_GOT32:
@@ -10746,6 +10780,7 @@ tc_gen_reloc (asection *section ATTRIBUTE_UNUSED, fixS *fixp)
          code = fixp->fx_r_type;
          break;
        }
+      /* Fall through.  */
     default:
       if (fixp->fx_pcrel)
        {
index 5afbb6f313c2d3c03e10c90e68ac872af30b39fc..70ac3798d85f2872bcd5ecd8661b0982d65a4cfd 100644 (file)
@@ -1041,7 +1041,7 @@ parse_memop (memS *memP,  /* Where to put the results.  */
     case I_BIT:
       /* Treat missing displacement as displacement of 0.  */
       mode |= D_BIT;
-      /* Fall into next case.  */
+      /* Fall through.  */
     case D_BIT | A_BIT | I_BIT:
     case D_BIT | I_BIT:
       /* Set MEMB bit in mode, and OR in mode bits.  */
@@ -1676,6 +1676,7 @@ md_assemble (char *textP)
              mem_fmt (args, oP, 1);
              break;
            }
+         /* Fall through.  */
        case MEM2:
        case MEM4:
        case MEM8:
index 83ea813792aac768077782d58ed85478e910e890..d6c2822b3bfee145249f8fd5bf47061edabe3762 100644 (file)
@@ -2738,6 +2738,7 @@ slot_index (unsigned long slot_addr,
                as_fatal (_("Only constant offsets are supported"));
                break;
              }
+           /* Fall through.  */
          case rs_fill:
            s_index += 3 * (first_frag->fr_offset >> 4);
            break;
@@ -5635,6 +5636,7 @@ operand_match (const struct ia64_opcode *idesc, int res_index, expressionS *e)
       /* SOR must be an integer multiple of 8 */
       if (e->X_op == O_constant && e->X_add_number & 0x7)
        return OPERAND_OUT_OF_RANGE;
+      /* Fall through.  */
     case IA64_OPND_SOF:
     case IA64_OPND_SOL:
       if (e->X_op == O_constant)
@@ -5790,6 +5792,7 @@ operand_match (const struct ia64_opcode *idesc, int res_index, expressionS *e)
     case IA64_OPND_IMM14:
     case IA64_OPND_IMM22:
       relocatable = 1;
+      /* Fall through.  */
     case IA64_OPND_IMM1:
     case IA64_OPND_IMM8:
     case IA64_OPND_IMM8U4:
@@ -5929,6 +5932,7 @@ operand_match (const struct ia64_opcode *idesc, int res_index, expressionS *e)
          ++CURR_SLOT.num_fixups;
          return OPERAND_MATCH;
        }
+      /* Fall through.  */
     case IA64_OPND_TAG13:
     case IA64_OPND_TAG13b:
       switch (e->X_op)
@@ -9281,6 +9285,7 @@ dep->name, idesc->name, (rsrc_write?"write":"read"), note)
                    {
                      specs[count++] = tmpl;
                    }
+                 /* Fall through.  */
                case AR_RSC:
                  if (!rsrc_write &&
                      (regno == AR_BSPSTORE
@@ -10263,7 +10268,7 @@ remove_marked_resource (struct rsrc *rs)
     case IA64_DVS_SPECIFIC:
       if (md.debug_dv)
        fprintf (stderr, "Implementation-specific, assume worst case...\n");
-      /* ...fall through...  */
+      /* Fall through.  */
     case IA64_DVS_INSTR:
       if (md.debug_dv)
        fprintf (stderr, "Inserting instr serialization\n");
index 2f80e0adb23f3a437d7c0bd3a2946e65612daeca..1fc52b1762c38dd0afeda17d7744bd66d11d9c8e 100644 (file)
@@ -2287,6 +2287,7 @@ build_indexed_byte (operand *op, int format ATTRIBUTE_UNUSED, int move_insn)
 
            default:
              as_bad (_("Invalid accumulator register."));
+             /* Fall through.  */
 
            case REG_D:
              byte = 0xE6;
index 4d2c0e4d05617ad017f3fa0100d6972ccb97ade8..4f64f5cf6f97240e19b623ad349c5be8ba2cf9a2 100644 (file)
@@ -3003,7 +3003,7 @@ m68k_ip (char *instring)
                                TAB (ABSTOPCREL, SZ_UNDEF));
                      break;
                    }
-                 /* Fall through into long.  */
+                 /* Fall through.  */
                case SIZE_LONG:
                  if (isvar (&opP->disp))
                    add_fix ('l', &opP->disp, 0, 0);
@@ -3109,6 +3109,7 @@ m68k_ip (char *instring)
              break;
            case '3':
              tmpreg &= 0xFF;
+             /* Fall through.  */
            case '8':
            case 'C':
            case 'j':
index 468a01b3f35025f6ca6ea52ca2edd9dc9d0ed436..ec03a296d2a64f6ed93a712d8b4aa34e9581dcef 100644 (file)
@@ -979,7 +979,7 @@ md_assemble (char * str)
          as_bad (_("M340 specific opcode used when assembling for M210"));
          break;
        }
-      /* drop through...  */
+      /* Fall through.  */
     case O2:
       op_end = parse_reg (op_end + 1, & reg);
       inst |= reg;
index f018f742747cea708804835395939546b0febef6..dac10c941db0799ab9af8d6795d0c7e0946e56b6 100644 (file)
@@ -1708,7 +1708,7 @@ md_convert_frag (bfd *abfd  ATTRIBUTE_UNUSED,
            operand = MEP_OPERAND_PCREL17A2;
            break;
          }
-       /* ...FALLTHROUGH... */
+       /* Fall through.  */
 
       case MEP_INSN_JMP:
        addend = target_address_for (fragP);
@@ -1723,6 +1723,7 @@ md_convert_frag (bfd *abfd  ATTRIBUTE_UNUSED,
 
       case MEP_INSN_BNEZ:
        bit = 1;
+       /* Fall through.  */
       case MEP_INSN_BEQZ:
        fragP->fr_opcode[1^e] = bit | (addend & 0xfe);
        operand = MEP_OPERAND_PCREL8A2;
@@ -1730,6 +1731,7 @@ md_convert_frag (bfd *abfd  ATTRIBUTE_UNUSED,
 
       case MEP_INSN_BNEI:
        bit = 4;
+       /* Fall through.  */
       case MEP_INSN_BEQI:
        if (subtype_mappings[fragP->fr_subtype].growth)
          {
index e9f86876dd25bb80c148d1bef663c3c35d2fbaf4..b56f9d21a7bd46d11bc9610458ac055a732f9b9b 100644 (file)
@@ -2683,6 +2683,7 @@ parse_alu (const char *line, metag_insn *insn,
              insn->bits |= (1 << 7);
              break;
            }
+         /* Fall through.  */
        default:
          as_bad (_("invalid quickrot register specified"));
          return NULL;
index b4985f09daecb190ab270c2464b794a338b53041..286efd4d25b6cc5d47a60c10181476e9c39cd35e 100644 (file)
@@ -2132,6 +2132,7 @@ md_apply_fix (fixS *   fixP,
     case BFD_RELOC_MICROBLAZE_64_TLSGD:
     case BFD_RELOC_MICROBLAZE_64_TLSLD:
       S_SET_THREAD_LOCAL (fixP->fx_addsy);
+      /* Fall through.  */
 
     case BFD_RELOC_MICROBLAZE_64_GOTPC:
     case BFD_RELOC_MICROBLAZE_64_GOT:
index 224b9d021fcff91a99f40bc8a06bd1f5a87f4a54..283ed80d3c5b0e400834e3f5dea664e3293162e3 100644 (file)
@@ -9848,6 +9848,7 @@ macro (struct mips_cl_insn *ip, char *str)
     {
     case M_DABS:
       dbl = 1;
+      /* Fall through.  */
     case M_ABS:
       /*    bgez    $a0,1f
            move    v0,$a0
@@ -9997,6 +9998,7 @@ macro (struct mips_cl_insn *ip, char *str)
 
     case M_BGEL:
       likely = 1;
+      /* Fall through.  */
     case M_BGE:
       if (op[1] == 0)
        macro_build_branch_rs (likely ? M_BGEZL : M_BGEZ, &offset_expr, op[0]);
@@ -10022,6 +10024,7 @@ macro (struct mips_cl_insn *ip, char *str)
 
     case M_BGTL_I:
       likely = 1;
+      /* Fall through.  */
     case M_BGT_I:
       /* Check for > max integer.  */
       if (imm_expr.X_add_number >= GPR_SMAX)
@@ -10068,6 +10071,7 @@ macro (struct mips_cl_insn *ip, char *str)
 
     case M_BGEUL:
       likely = 1;
+      /* Fall through.  */
     case M_BGEU:
       if (op[1] == 0)
        goto do_true;
@@ -10085,6 +10089,7 @@ macro (struct mips_cl_insn *ip, char *str)
 
     case M_BGTUL_I:
       likely = 1;
+      /* Fall through.  */
     case M_BGTU_I:
       if (op[0] == 0
          || (GPR_SIZE == 32
@@ -10112,6 +10117,7 @@ macro (struct mips_cl_insn *ip, char *str)
 
     case M_BGTL:
       likely = 1;
+      /* Fall through.  */
     case M_BGT:
       if (op[1] == 0)
        macro_build_branch_rs (likely ? M_BGTZL : M_BGTZ, &offset_expr, op[0]);
@@ -10128,6 +10134,7 @@ macro (struct mips_cl_insn *ip, char *str)
 
     case M_BGTUL:
       likely = 1;
+      /* Fall through.  */
     case M_BGTU:
       if (op[1] == 0)
        macro_build_branch_rsrt (likely ? M_BNEL : M_BNE,
@@ -10145,6 +10152,7 @@ macro (struct mips_cl_insn *ip, char *str)
 
     case M_BLEL:
       likely = 1;
+      /* Fall through.  */
     case M_BLE:
       if (op[1] == 0)
        macro_build_branch_rs (likely ? M_BLEZL : M_BLEZ, &offset_expr, op[0]);
@@ -10161,6 +10169,7 @@ macro (struct mips_cl_insn *ip, char *str)
 
     case M_BLEL_I:
       likely = 1;
+      /* Fall through.  */
     case M_BLE_I:
       if (imm_expr.X_add_number >= GPR_SMAX)
        goto do_true;
@@ -10185,6 +10194,7 @@ macro (struct mips_cl_insn *ip, char *str)
 
     case M_BLEUL:
       likely = 1;
+      /* Fall through.  */
     case M_BLEU:
       if (op[1] == 0)
        macro_build_branch_rsrt (likely ? M_BEQL : M_BEQ,
@@ -10202,6 +10212,7 @@ macro (struct mips_cl_insn *ip, char *str)
 
     case M_BLEUL_I:
       likely = 1;
+      /* Fall through.  */
     case M_BLEU_I:
       if (op[0] == 0
          || (GPR_SIZE == 32
@@ -10229,6 +10240,7 @@ macro (struct mips_cl_insn *ip, char *str)
 
     case M_BLTL:
       likely = 1;
+      /* Fall through.  */
     case M_BLT:
       if (op[1] == 0)
        macro_build_branch_rs (likely ? M_BLTZL : M_BLTZ, &offset_expr, op[0]);
@@ -10245,6 +10257,7 @@ macro (struct mips_cl_insn *ip, char *str)
 
     case M_BLTUL:
       likely = 1;
+      /* Fall through.  */
     case M_BLTU:
       if (op[1] == 0)
        goto do_false;
@@ -10262,11 +10275,13 @@ macro (struct mips_cl_insn *ip, char *str)
 
     case M_DDIV_3:
       dbl = 1;
+      /* Fall through.  */
     case M_DIV_3:
       s = "mflo";
       goto do_div3;
     case M_DREM_3:
       dbl = 1;
+      /* Fall through.  */
     case M_REM_3:
       s = "mfhi";
     do_div3:
@@ -10458,11 +10473,13 @@ macro (struct mips_cl_insn *ip, char *str)
 
     case M_DLCA_AB:
       dbl = 1;
+      /* Fall through.  */
     case M_LCA_AB:
       call = 1;
       goto do_la;
     case M_DLA_AB:
       dbl = 1;
+      /* Fall through.  */
     case M_LA_AB:
     do_la:
       /* Load the address of a symbol into a register.  If breg is not
@@ -12685,6 +12702,7 @@ macro (struct mips_cl_insn *ip, char *str)
 
     case M_DMUL:
       dbl = 1;
+      /* Fall through.  */
     case M_MUL:
       if (mips_opts.arch == CPU_R5900)
        macro_build (NULL, dbl ? "dmultu" : "multu", "d,s,t", op[0], op[1],
@@ -12698,6 +12716,7 @@ macro (struct mips_cl_insn *ip, char *str)
 
     case M_DMUL_I:
       dbl = 1;
+      /* Fall through.  */
     case M_MUL_I:
       /* The MIPS assembler some times generates shifts and adds.  I'm
         not trying to be that fancy. GCC should do this for us
@@ -12710,12 +12729,14 @@ macro (struct mips_cl_insn *ip, char *str)
 
     case M_DMULO_I:
       dbl = 1;
+      /* Fall through.  */
     case M_MULO_I:
       imm = 1;
       goto do_mulo;
 
     case M_DMULO:
       dbl = 1;
+      /* Fall through.  */
     case M_MULO:
     do_mulo:
       start_noreorder ();
@@ -12747,12 +12768,14 @@ macro (struct mips_cl_insn *ip, char *str)
 
     case M_DMULOU_I:
       dbl = 1;
+      /* Fall through.  */
     case M_MULOU_I:
       imm = 1;
       goto do_mulou;
 
     case M_DMULOU:
       dbl = 1;
+      /* Fall through.  */
     case M_MULOU:
     do_mulou:
       start_noreorder ();
@@ -13456,11 +13479,13 @@ mips16_macro (struct mips_cl_insn *ip)
 
     case M_DDIV_3:
       dbl = 1;
+      /* Fall through.  */
     case M_DIV_3:
       s = "mflo";
       goto do_div3;
     case M_DREM_3:
       dbl = 1;
+      /* Fall through.  */
     case M_REM_3:
       s = "mfhi";
     do_div3:
@@ -13505,6 +13530,7 @@ mips16_macro (struct mips_cl_insn *ip)
 
     case M_DMUL:
       dbl = 1;
+      /* Fall through.  */
     case M_MUL:
       macro_build (NULL, dbl ? "dmultu" : "multu", "x,y", op[1], op[2]);
       macro_build (NULL, "mflo", "x", op[0]);
index a0db7690ace41741e97ed1cb25d0c4e07999f054..570916db6e9fd659e4680d149decd79c439c5a6b 100644 (file)
@@ -432,6 +432,7 @@ addr_mode (char *operand,
                  addrmodeP->disp[0] = str + 2;
                  return -1;
                }
+             /* Fall through.  */
            default:
              as_bad (_("Invalid syntax in PC-relative addressing mode"));
              return 0;
@@ -481,7 +482,7 @@ addr_mode (char *operand,
        {
        case 'f':
          addrmodeP->float_flag = 1;
-         /* Drop through.  */
+         /* Fall through.  */
        case 'r':
          if (str[1] >= '0' && str[1] < '8')
            {
@@ -564,7 +565,7 @@ addr_mode (char *operand,
                  str[strl - 4] = 0;
                  return -1;            /* reg rel */
                }
-             /* Drop through.  */
+             /* Fall through.  */
 
            default:
              if (!strncmp (&str[strl - 4], "(fp", 3))
@@ -925,6 +926,7 @@ encode_operand (int argc,
        case 'f':               /* Operand of sfsr turns out to be a nasty
                                   specialcase.  */
          opcode_bit_ptr -= 5;
+         /* Fall through.  */
        case 'Z':               /* Float not immediate.  */
        case 'F':               /* 32 bit float general form.  */
        case 'L':               /* 64 bit float.  */
index bb0f689cf1ba2cceb18c0c8153165d3519ba6d10..07dccc6e83d51898aca63f0f74c25cfea3ca4670 100644 (file)
@@ -2401,8 +2401,10 @@ md_apply_fix (struct fix * f ATTRIBUTE_UNUSED,
 
     case BFD_RELOC_RX_GPRELL:
       val >>= 1;
+      /* Fall through.  */
     case BFD_RELOC_RX_GPRELW:
       val >>= 1;
+      /* Fall through.  */
     case BFD_RELOC_RX_GPRELB:
 #if RX_OPCODE_BIG_ENDIAN
       op[1] = val & 0xff;
index f96879b3ab1beae1a5a25f4950cde03f9fa8ff00..f68cc6fb616e503cb2ad6bab826cbfc2002ffa92 100644 (file)
@@ -7453,6 +7453,7 @@ s3_gen_reloc (asection * section ATTRIBUTE_UNUSED, fixS * fixp)
           code = BFD_RELOC_32_PCREL;
           break;
         }
+      /* Fall through.  */
     case BFD_RELOC_HI16_S:
     case BFD_RELOC_LO16:
     case BFD_RELOC_SCORE_JMP:
index 941467be3bfcaf13317c9df8ed4c7a08ff28f211..d7dd108d6df3ad296cc71806d94cb2059b5e67d8 100644 (file)
@@ -6930,6 +6930,7 @@ s7_gen_reloc (asection * section ATTRIBUTE_UNUSED, fixS * fixp)
           code = BFD_RELOC_32_PCREL;
           break;
         }
+      /* Fall through.  */
     case BFD_RELOC_HI16_S:
     case BFD_RELOC_LO16:
     case BFD_RELOC_SCORE_JMP:
index f65fbb22de4f17d8d04ad6d926956836269e8b60..55966e0376cdd0423e8b6d5a21d1dbc610bf06c3 100644 (file)
@@ -2414,11 +2414,13 @@ build_Mytes (sh_opcode_info *opcode, sh_operand_info *operand)
              break;
            case IMM0_3s:
              nbuf[indx] |= 0x08;
+             /* Fall through.  */
            case IMM0_3c:
              insert (output + low_byte, BFD_RELOC_SH_IMM3, 0, operand);
              break;
            case IMM0_3Us:
              nbuf[indx] |= 0x80;
+             /* Fall through.  */
            case IMM0_3Uc:
              insert (output + low_byte, BFD_RELOC_SH_IMM3U, 0, operand);
              break;
index 84c9ece720ae4e2c28f09cc26d334bd6c7feae5c..f86a5352c609a9ec06c92c9abfe0b7e5be048323 100644 (file)
@@ -2638,9 +2638,11 @@ md_apply_fix (fixS *fixP, valueT *value, segT seg ATTRIBUTE_UNUSED)
     {
     case BFD_RELOC_32:
       buf[3] = val >> 24;
+      /* Fall through.  */
     case BFD_RELOC_24:
     case BFD_RELOC_24_PCREL:
       buf[2] = val >> 16;
+      /* Fall through.  */
     case BFD_RELOC_16:
     case BFD_RELOC_16_PCREL:
     case BFD_RELOC_LO16:
@@ -2718,24 +2720,28 @@ md_parse_option (int c, const char *arg)
 
     case 'b':
       as_warn (_("Option -b is depreciated, please use -mbig"));
+      /* Fall through.  */
     case OPTION_BIG:             /* big model */
       tic4x_big_model = 1;
       break;
 
     case 'p':
       as_warn (_("Option -p is depreciated, please use -mmemparm"));
+      /* Fall through.  */
     case OPTION_MEMPARM:         /* push args */
       tic4x_reg_args = 0;
       break;
 
     case 'r':
       as_warn (_("Option -r is depreciated, please use -mregparm"));
+      /* Fall through.  */
     case OPTION_REGPARM:        /* register args */
       tic4x_reg_args = 1;
       break;
 
     case 's':
       as_warn (_("Option -s is depreciated, please use -msmall"));
+      /* Fall through.  */
     case OPTION_SMALL:         /* small model */
       tic4x_big_model = 0;
       break;
index b85b5bb91bd56e2206b98659d4ae3616afec84e3..e358c842cfac152515c245e48accbeeafc967585 100644 (file)
@@ -1796,8 +1796,10 @@ vip_op (char *optext, struct vop *vopP)
        {
        case 'l':
          mode += 2;
+         /* Fall through.  */
        case 'w':
          mode += 2;
+         /* Fall through.  */
        case ' ':       /* Assumed B^ until our caller changes it.  */
        case 'b':
          break;
index 05e161615a3a386d71e203e1834fd1a42e575f1f..5b1d255aa6f24e5e32c3dd949b683cb42c0e70cf 100644 (file)
@@ -1520,8 +1520,7 @@ md_assemble (char *str0)
          return;
        }
       this_dest = r1;
-
-      /* fall through...  */
+      /* Fall through.  */
 
     case mode_i:
       /* MOVIL/WRTL traditionally get an implicit "%l" applied
index 887160db01de1ff0a6ef5f6ef6127b34315cca8f..ca6793e7e1bebc70ceaded5aa6b1154fbcc9d94f 100644 (file)
@@ -382,6 +382,7 @@ md_cgen_lookup_reloc (const CGEN_INSN *    insn ATTRIBUTE_UNUSED,
 
     case XSTORMY16_OPERAND_REL8_4:
       fixP->fx_addnumber -= 2;
+      /* Fall through.  */
     case XSTORMY16_OPERAND_REL8_2:
       fixP->fx_addnumber -= 2;
       fixP->fx_pcrel = 1;
@@ -389,7 +390,7 @@ md_cgen_lookup_reloc (const CGEN_INSN *    insn ATTRIBUTE_UNUSED,
 
     case XSTORMY16_OPERAND_REL12:
       fixP->fx_where += 2;
-      /* Fall through...  */
+      /* Fall through.  */
     case XSTORMY16_OPERAND_REL12A:
       fixP->fx_addnumber -= 2;
       fixP->fx_pcrel = 1;
index dcc74f6ed58a60c310551ebd27926fe143fe87bc..b7e075a80949d164a1cbfa726847325e7144a0b1 100644 (file)
@@ -317,6 +317,7 @@ z80_start_line_hook (void)
              *p++ = buf[2];
              break;
            }
+         /* Fall through.  */
        case '"':
          for (quote = *p++; quote != *p && '\n' != *p; ++p)
            /* No escapes.  */ ;
@@ -874,6 +875,7 @@ emit_mr (char prefix, char opcode, const char *args)
            }
          check_mach (INS_UNPORT);
        }
+      /* Fall through.  */
     case O_register:
       emit_mx (prefix, opcode, 0, & arg_m);
       break;
@@ -1776,6 +1778,7 @@ emit_mulub (char prefix ATTRIBUTE_UNUSED, char opcode, const char * args)
              *q = opcode + ((reg - 'b') << 3);
              break;
            }
+         /* Fall through.  */
        default:
          ill_op ();
        }
index 9b6ee3867a5761f790938db9db198a3635e8887d..5be09fd75815e41b78076a2000fe7f404795d9c1 100644 (file)
@@ -1000,11 +1000,14 @@ apply_fix (char *ptr, bfd_reloc_code_real_type type, expressionS *operand,
       *ptr++ = n >> 24;
       *ptr++ = n >> 20;
       *ptr++ = n >> 16;
+      /* Fall through.  */
     case 4:                    /* 4 nibbles == 16 bits.  */
       *ptr++ = n >> 12;
       *ptr++ = n >> 8;
+      /* Fall through.  */
     case 2:
       *ptr++ = n >> 4;
+      /* Fall through.  */
     case 1:
       *ptr++ = n >> 0;
       break;
@@ -1152,7 +1155,7 @@ build_bytes (opcode_entry_type *this_try, struct z8k_op *operand ATTRIBUTE_UNUSE
               /*case ARG_IMMNMINUS1: not used.  */
              case ARG_IMM4M1:
                imm_operand->X_add_number--;
-                /* Drop through.  */
+                /* Fall through.  */
              case ARG_IMM4:
                 if (imm_operand->X_add_number > 15)
                  as_bad (_("immediate value out of range"));
@@ -1160,7 +1163,7 @@ build_bytes (opcode_entry_type *this_try, struct z8k_op *operand ATTRIBUTE_UNUSE
                break;
              case ARG_NIM8:
                imm_operand->X_add_number = -imm_operand->X_add_number;
-                /* Drop through.  */
+                /* Fall through.  */
              case ARG_IMM8:
                output_ptr = apply_fix (output_ptr, BFD_RELOC_8, imm_operand, 2);
                break;
index 7203e59746f0d11e49340f263712a6c150dbca2f..8b57a679d4863e2be7566873125ca7d42136f6af 100644 (file)
@@ -121,8 +121,8 @@ quote_string_for_make (FILE *file, const char *src)
          if (file)
            putc (c, file);
          i++;
-         /* Fall through.  This can mishandle things like "$(" but
-            there's no easy fix.  */
+         /* Fall through.  */
+         /* This can mishandle things like "$(" but there's no easy fix.  */
        default:
        ordinary_char:
          /* This can mishandle characters in the string "\0\n%*?[\\~";
index 4fbdf4248e2e38a00bb8f0fc5bd3e9addc0b52a0..7294f179964d0d2eb04ce9d2326b07dcc439a7ee 100644 (file)
@@ -1177,6 +1177,7 @@ dot_cfi_val_encoded_addr (int ignored ATTRIBUTE_UNUSED)
     case O_constant:
       if ((encoding & 0x70) != DW_EH_PE_pcrel)
        break;
+      /* Fall through.  */
     default:
       encoding = DW_EH_PE_omit;
       break;
index b1cdb38bab58add5550d1affbbbfcd363b298279..b9983d86b688e3d7d3112624e655d2ed8e14412d 100644 (file)
@@ -1008,6 +1008,7 @@ operand (expressionS *expressionP, enum expr_mode mode)
       /* '~' is permitted to start a label on the Delta.  */
       if (is_name_beginner (c))
        goto isname;
+      /* Fall through.  */
     case '!':
     case '-':
     case '+':
index 6e6fd67026b97bb632e7a235814803adb858a5d1..a1e821c12d8c125957611f2869b7411f1c2b66de 100644 (file)
@@ -1,3 +1,7 @@
+2016-10-06  Alan Modra  <amodra@gmail.com>
+
+       * gprof.c: Add missing fall through comments.
+
 2016-09-26  Vlad Zakharov  <vzakhar@synopsys.com>
 
        * Makefile.in: Regenerate.
index 4df99751a3ea99f9a43fd55a75f5b12e2df69464..43e209096c9457209ba63872c3999cdb051d4ee9 100644 (file)
@@ -259,11 +259,13 @@ main (int argc, char **argv)
          break;
        case 'E':
          sym_id_add (optarg, EXCL_TIME);
+         /* Fall through.  */
        case 'e':
          sym_id_add (optarg, EXCL_GRAPH);
          break;
        case 'F':
          sym_id_add (optarg, INCL_TIME);
+         /* Fall through.  */
        case 'f':
          sym_id_add (optarg, INCL_GRAPH);
          break;
index 3660d87c9159c69fcf92006be5768201d9ee888f..40d6c1455c3623aa8a0b2b4aaac73b10fb634a9b 100644 (file)
@@ -1,3 +1,8 @@
+2016-10-06  Alan Modra  <amodra@gmail.com>
+
+       * lexsup.c: Spell fall through comments consistently and add
+       missing fall through comments.
+
 2016-10-06  Alan Modra  <amodra@gmail.com>
 
        * plugin.c (asymbol_from_plugin_symbol): Avoid compiler warning
index 294c4b2d1b9fb481899fadf2f5d2a163fcd6ff14..d39f8489ae0a87fbb5a6d942de61e15f31a29f7b 100644 (file)
@@ -1280,7 +1280,7 @@ parse_args (unsigned argc, char **argv)
          break;
        case OPTION_TASK_LINK:
          link_info.task_link = TRUE;
-         /* Fall through - do an implied -r option.  */
+         /* Fall through.  */
        case OPTION_UR:
          if (bfd_link_pic (&link_info))
            einfo (_("%P%F: -r and %s may not be used together\n"),
@@ -1619,6 +1619,7 @@ parse_args (unsigned argc, char **argv)
       break;
     case dynamic_list_data:
       link_info.dynamic_data = TRUE;
+      /* Fall through.  */
     case dynamic_list:
       link_info.dynamic = TRUE;
       break;
index 660276822e8e0c1ebc0813da937e260fb802fc28..441665914b16b5cf129d1da530a83060c4302b07 100644 (file)
@@ -1,3 +1,20 @@
+2016-10-06  Alan Modra  <amodra@gmail.com>
+
+       * aarch64-opc.c: Spell fall through comments consistently.
+       * i386-dis.c: Likewise.
+       * aarch64-dis.c: Add missing fall through comments.
+       * aarch64-opc.c: Likewise.
+       * arc-dis.c: Likewise.
+       * arm-dis.c: Likewise.
+       * i386-dis.c: Likewise.
+       * m68k-dis.c: Likewise.
+       * mep-asm.c: Likewise.
+       * ns32k-dis.c: Likewise.
+       * sh-dis.c: Likewise.
+       * tic4x-dis.c: Likewise.
+       * tic6x-dis.c: Likewise.
+       * vax-dis.c: Likewise.
+
 2016-10-06  Alan Modra  <amodra@gmail.com>
 
        * arc-ext.c (create_map): Add missing break.
index 82afe8b8f5b2340b2305fd577582c7f18696f9c3..d8c5fa7aa5ca86fd477579c72b054f5c6ed946e0 100644 (file)
@@ -788,10 +788,15 @@ decode_limm (uint32_t esize, aarch64_insn value, int64_t *result)
   switch (simd_size)
     {
     case  2: imm = (imm <<  2) | imm;
+      /* Fall through.  */
     case  4: imm = (imm <<  4) | imm;
+      /* Fall through.  */
     case  8: imm = (imm <<  8) | imm;
+      /* Fall through.  */
     case 16: imm = (imm << 16) | imm;
+      /* Fall through.  */
     case 32: imm = (imm << 32) | imm;
+      /* Fall through.  */
     case 64: break;
     default: assert (0); return 0;
     }
index adc34e29b9c70b6d18cc4b014f8bd6dbaae205c6..4b1411f4ba6e1cbd6c78bacffffd4daaecf69d3c 100644 (file)
@@ -1156,10 +1156,15 @@ build_immediate_table (void)
            switch (log_e)
              {
              case 1: imm = (imm <<  2) | imm;
+               /* Fall through.  */
              case 2: imm = (imm <<  4) | imm;
+               /* Fall through.  */
              case 3: imm = (imm <<  8) | imm;
+               /* Fall through.  */
              case 4: imm = (imm << 16) | imm;
+               /* Fall through.  */
              case 5: imm = (imm << 32) | imm;
+               /* Fall through.  */
              case 6: break;
              default: abort ();
              }
@@ -1969,7 +1974,7 @@ operand_general_constraint_met_p (const aarch64_opnd_info *opnds, int idx,
                {
                case OP_MOV_IMM_WIDEN:
                  imm = ~imm;
-                 /* Fall through...  */
+                 /* Fall through.  */
                case OP_MOV_IMM_WIDE:
                  if (!aarch64_wide_constant_p (imm, esize == 4, NULL))
                    {
index e4df678dfa532f403ea686a4f0361c75610d1113..c4e118e34051b650a22a784b4b661328d190b470 100644 (file)
@@ -703,6 +703,7 @@ arc_insn_length (bfd_byte msb, bfd_byte lsb, struct disassemble_info *info)
           if (minor_opcode < 4)
             return 2;
         }
+      /* Fall through.  */
     case bfd_mach_arc_arc600:
       return (major_opcode > 0xb) ? 2 : 4;
       break;
index fc9ac610aaef349adb882bd94fd7a614b6ff6d6c..87d4930b30fd9fef9880dbe803abaafc583c27a2 100644 (file)
@@ -4686,6 +4686,7 @@ print_insn_arm (bfd_vma pc, struct disassemble_info *info, long given)
 
                    case 'S':
                      allow_unpredictable = TRUE;
+                     /* Fall through.  */
                    case 's':
                       if ((given & 0x004f0000) == 0x004f0000)
                        {
index c08109e00a7f8ad6de21d5951c70bab71e9414d3..5c5f926d30d35538e18c51941a2aa9028e115dae 100644 (file)
@@ -14850,13 +14850,14 @@ intel_operand_size (int bytemode, int sizeflag)
          oappend ("QWORD PTR ");
          break;
        }
+      /* Fall through.  */
     case stack_v_mode:
       if (address_mode == mode_64bit && ((sizeflag & DFLAG) || (rex & REX_W)))
        {
          oappend ("QWORD PTR ");
          break;
        }
-      /* FALLTHRU */
+      /* Fall through.  */
     case v_mode:
     case v_swap_mode:
     case dq_mode:
@@ -15233,6 +15234,7 @@ OP_E_register (int bytemode, int sizeflag)
          names = names64;
          break;
        }
+      /* Fall through.  */
     case stack_v_mode:
       if (address_mode == mode_64bit && ((sizeflag & DFLAG) || (rex & REX_W)))
        {
@@ -15240,7 +15242,7 @@ OP_E_register (int bytemode, int sizeflag)
          break;
        }
       bytemode = v_mode;
-      /* FALLTHRU */
+      /* Fall through.  */
     case v_mode:
     case v_swap_mode:
     case dq_mode:
@@ -15321,7 +15323,7 @@ OP_E_memory (int bytemode, int sizeflag)
              shift = vex.w ? 3 : 2;
              break;
            }
-         /* Fall through if vex.b == 0.  */
+         /* Fall through.  */
        case xmmqd_mode:
        case xmmdw_mode:
        case ymmq_mode:
@@ -17301,6 +17303,7 @@ get_vex_imm8 (int sizeflag, int opnum)
                    if (base != 5)
                      /* No displacement. */
                      break;
+                   /* Fall through.  */
                  case 2:
                    /* 4 byte displacement.  */
                    bytes_before_imm += 4;
@@ -17327,6 +17330,7 @@ get_vex_imm8 (int sizeflag, int opnum)
                  if (modrm.rm != 6)
                    /* No displacement. */
                    break;
+                 /* Fall through.  */
                case 2:
                  /* 2 byte displacement.  */
                  bytes_before_imm += 2;
index 11c057d61394edc371814ace91d2ae37c3257e29..1e7c830292314b988f5909c445644e56438d8fca 100644 (file)
@@ -1220,6 +1220,7 @@ print_insn_arg (const char *d,
 
     case 'X':
       place = '8';
+      /* Fall through.  */
     case 'Y':
     case 'Z':
     case 'W':
index d8a1cf307f0d0d06b0579148d13fcd8020fd179d..89116ee8e8e004f5234205dbf6e74ab3c6e653d0 100644 (file)
@@ -745,6 +745,7 @@ expand_string (const char *in, int first_only)
                  break;
                case '(':
                  depth++;
+                 /* Fall through.  */
                default:
                  args[narg].len++;
                  break;
index dd52cffc0795294fbb53394e6cb5afa8f0dd467e..b14d5d747725a535c0e412117279ea6b081a2102 100644 (file)
@@ -472,6 +472,7 @@ print_insn_arg (int d,
     case 'f':
       /* A "gen" operand but 5 bits from the end of instruction.  */
       ioffset -= 5;
+      /* Fall through.  */
     case 'Z':
     case 'F':
     case 'L':
index 1f8dc8a5ac388b4c1bc6cc3fcab374b070f2f929..7af514b8ea57ccfeef5dc3b92f933eabf40e2535 100644 (file)
@@ -836,6 +836,7 @@ print_insn_sh (bfd_vma memaddr, struct disassemble_info *info)
                  fprintf_fn (stream, "xd%d", rn & ~1);
                  break;
                }
+             /* Fall through.  */
            case D_REG_N:
              fprintf_fn (stream, "dr%d", rn);
              break;
@@ -845,6 +846,7 @@ print_insn_sh (bfd_vma memaddr, struct disassemble_info *info)
                  fprintf_fn (stream, "xd%d", rm & ~1);
                  break;
                }
+             /* Fall through.  */
            case D_REG_M:
              fprintf_fn (stream, "dr%d", rm);
              break;
index c35e050397f86d79fc35e8ea363ead640aa52247..92c9dcb4cb36145ed7ed3708cfa89e8d332ce7ce 100644 (file)
@@ -467,6 +467,7 @@ tic4x_print_op (struct disassemble_info *info,
                 return 0;
               break;
             }
+         /* Fall through.  */
 
        case 'J': /* Indirect (short) 8--15.  */
          if (! tic4x_print_indirect (info, INDIRECT_SHORT,
index ec1dcc16465ed302dbcff98c5e34d366d944433e..f6426566f5826daea04b20dd593199f6c842301c 100644 (file)
@@ -1075,6 +1075,7 @@ print_insn_tic6x (bfd_vma addr, struct disassemble_info *info)
                case tic6x_coding_mem_offset_minus_one_noscale:
                case tic6x_coding_mem_offset_minus_one:
                  fld_val += 1;
+                 /* Fall through.  */
                case tic6x_coding_mem_offset_noscale:
                case tic6x_coding_mem_offset:
                  mem_offset = fld_val;
index 937c45206b89c8add6bbd179c5cffb88aaf8b60c..0aec8122911439f12bf1233de29599c1f2a41918 100644 (file)
@@ -296,6 +296,7 @@ print_insn_mode (const char *d,
       break;
     case 0xB0: /* Displacement byte deferred:  *displ(Rn).  */
       (*info->fprintf_func) (info->stream, "*");
+      /* Fall through.  */
     case 0xA0: /* Displacement byte:           displ(Rn).  */
       if (reg == 0xF)
        (*info->print_address_func) (addr + 2 + NEXTBYTE (p), info);
@@ -305,6 +306,7 @@ print_insn_mode (const char *d,
       break;
     case 0xD0: /* Displacement word deferred:  *displ(Rn).  */
       (*info->fprintf_func) (info->stream, "*");
+      /* Fall through.  */
     case 0xC0: /* Displacement word:           displ(Rn).  */
       if (reg == 0xF)
        (*info->print_address_func) (addr + 3 + NEXTWORD (p), info);
@@ -314,6 +316,7 @@ print_insn_mode (const char *d,
       break;
     case 0xF0: /* Displacement long deferred:  *displ(Rn).  */
       (*info->fprintf_func) (info->stream, "*");
+      /* Fall through.  */
     case 0xE0: /* Displacement long:           displ(Rn).  */
       if (reg == 0xF)
        (*info->print_address_func) (addr + 5 + NEXTLONG (p), info);