Fix [-Wjump-misses-init] in generated zend_opcode.c

Girgias · Girgias · commit 826a7456717b · 2020-04-18T14:52:49.000+02:00
diff --git a/Zend/zend_opcode.c b/Zend/zend_opcode.c
@@ -682,7 +682,8 @@ static void emit_live_range(
 				/* Trivial live-range, no need to store it. */
 				return;
 			}
-			/* break missing intentionally */
+		}
+		/* explicit fallthrough */
 		default:
 			start++;
 			kind = ZEND_LIVE_TMPVAR;
@@ -693,7 +694,6 @@ static void emit_live_range(
 				return;
 			}
 			break;
-		}
 		case ZEND_COPY_TMP:
 		{
 			/* COPY_TMP has a split live-range: One from the definition until the use in

Original file line number	Diff line number	Diff line change
`@@ -682,7 +682,8 @@ static void emit_live_range(`
`682`	`682`	`/* Trivial live-range, no need to store it. */`
`683`	`683`	`return;`
`684`	`684`	`}`
`685`		`- /* break missing intentionally */`
	`685`	`+ }`
	`686`	`+ /* explicit fallthrough */`
`686`	`687`	`default:`
`687`	`688`	`start++;`
`688`	`689`	`kind = ZEND_LIVE_TMPVAR;`
`@@ -693,7 +694,6 @@ static void emit_live_range(`
`693`	`694`	`return;`
`694`	`695`	`}`
`695`	`696`	`break;`
`696`		`- }`
`697`	`697`	`case ZEND_COPY_TMP:`
`698`	`698`	`{`
`699`	`699`	`/* COPY_TMP has a split live-range: One from the definition until the use in`