This is the mail archive of the binutils@sourceware.org mailing list for the binutils project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

[PATCH 4/4] fix fallthough statements to allow gcc-7 compile


Signed-off-by: Daniel Black <daniel.black@au.ibm.com>
---
 binutils/dwarf.c     | 3 +++
 gas/config/obj-elf.c | 1 +
 gas/depend.c         | 5 +++--
 gas/dw2gencfi.c      | 1 +
 gas/expr.c           | 1 +
 5 files changed, 9 insertions(+), 2 deletions(-)

diff --git a/binutils/dwarf.c b/binutils/dwarf.c
index 282e069..091921f 100644
--- a/binutils/dwarf.c
+++ b/binutils/dwarf.c
@@ -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:
@@ -7580,6 +7582,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;
diff --git a/gas/config/obj-elf.c b/gas/config/obj-elf.c
index 8af563f..4b85b52 100644
--- a/gas/config/obj-elf.c
+++ b/gas/config/obj-elf.c
@@ -813,6 +813,7 @@ obj_elf_parse_section_letters (char *str, size_t len, bfd_boolean *is_clone)
 		}
 	      break;
 	    }
+	  /* fallthrough */
 	default:
 	  {
 	    const char *bad_msg = _("unrecognized .section attribute:"
diff --git a/gas/depend.c b/gas/depend.c
index 7203e59..86d0566 100644
--- a/gas/depend.c
+++ b/gas/depend.c
@@ -121,8 +121,9 @@ 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.  */
+	  /* This can mishandle things like "$(" but
+	     there's no easy fix so */
+	  /* fall through */
 	default:
 	ordinary_char:
 	  /* This can mishandle characters in the string "\0\n%*?[\\~";
diff --git a/gas/dw2gencfi.c b/gas/dw2gencfi.c
index fb3e302..a362a57 100644
--- a/gas/dw2gencfi.c
+++ b/gas/dw2gencfi.c
@@ -1153,6 +1153,7 @@ dot_cfi_val_encoded_addr (int ignored ATTRIBUTE_UNUSED)
     case O_constant:
       if ((encoding & 0x70) != DW_EH_PE_pcrel)
 	break;
+      /* fallthrough */
     default:
       encoding = DW_EH_PE_omit;
       break;
diff --git a/gas/expr.c b/gas/expr.c
index b1cdb38..3611f2a 100644
--- a/gas/expr.c
+++ b/gas/expr.c
@@ -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;
+      /* fallthrough */
     case '!':
     case '-':
     case '+':
-- 
2.7.4


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]