Changeset 65651 in vbox for trunk/src/VBox/ValidationKit

Timestamp:

Feb 7, 2017 11:46:36 AM (8 years ago)

Author:

vboxsync

svn:sync-xref-src-repo-rev:

113337

Message:

gcc 7: Validation Kit: fall thru

Location:

trunk/src/VBox/ValidationKit

Files:

• bootsectors/bs3kit/VBoxBs3ObjConverter.cpp (modified) (1 diff)
• utils/cpu/cidet-core.cpp (modified) (4 diffs)

trunk/src/VBox/ValidationKit/bootsectors/bs3kit/VBoxBs3ObjConverter.cpp

@@ -4237 +4237 @@
             case OMF_LPUBDEF32:
                 pOmfStuff->fProbably32bit = true;
+                /* fall thru */
             case OMF_PUBDEF16:
             case OMF_LPUBDEF16:

trunk/src/VBox/ValidationKit/utils/cpu/cidet-core.cpp

@@ -1919 +1919 @@
     switch (pThis->pCurInstr->cbOpcode)
     {
-        case 3: off++;
-        case 2: off++;
+        case 3: off++; /* fall thru */
+        case 2: off++; /* fall thru */
         case 1: off++;
             break;
@@ -2037 +2037 @@
     switch (pThis->pCurInstr->cbOpcode)
     {
-        case 3: pThis->abInstr[off++] = *pbOpcode++;
-        case 2: pThis->abInstr[off++] = *pbOpcode++;
+        case 3: pThis->abInstr[off++] = *pbOpcode++; /* fall thru */
+        case 2: pThis->abInstr[off++] = *pbOpcode++; /* fall thru */
         case 1: pThis->abInstr[off++] = *pbOpcode++;
             break;
@@ -2059 +2059 @@
             {
                 case 0: break;
-                case 8: pThis->abInstr[off + 3] = (uDispValue >> 56) & UINT8_C(0xff);
-                case 7: pThis->abInstr[off + 3] = (uDispValue >> 48) & UINT8_C(0xff);
-                case 6: pThis->abInstr[off + 3] = (uDispValue >> 40) & UINT8_C(0xff);
-                case 5: pThis->abInstr[off + 3] = (uDispValue >> 32) & UINT8_C(0xff);
-                case 4: pThis->abInstr[off + 3] = (uDispValue >> 24) & UINT8_C(0xff);
-                case 3: pThis->abInstr[off + 2] = (uDispValue >> 16) & UINT8_C(0xff);
-                case 2: pThis->abInstr[off + 1] = (uDispValue >>  8) & UINT8_C(0xff);
+                case 8: pThis->abInstr[off + 3] = (uDispValue >> 56) & UINT8_C(0xff); /* fall thru */
+                case 7: pThis->abInstr[off + 3] = (uDispValue >> 48) & UINT8_C(0xff); /* fall thru */
+                case 6: pThis->abInstr[off + 3] = (uDispValue >> 40) & UINT8_C(0xff); /* fall thru */
+                case 5: pThis->abInstr[off + 3] = (uDispValue >> 32) & UINT8_C(0xff); /* fall thru */
+                case 4: pThis->abInstr[off + 3] = (uDispValue >> 24) & UINT8_C(0xff); /* fall thru */
+                case 3: pThis->abInstr[off + 2] = (uDispValue >> 16) & UINT8_C(0xff); /* fall thru */
+                case 2: pThis->abInstr[off + 1] = (uDispValue >>  8) & UINT8_C(0xff); /* fall thru */
                 case 1: pThis->abInstr[off] = uDispValue & UINT8_C(0xff);
                     break;
@@ -2084 +2084 @@
             switch (pThis->aOperands[iOp].cb)
             {
-                case 8: pThis->abInstr[off + 3] = (uImmValue >> 56) & UINT8_C(0xff);
-                case 7: pThis->abInstr[off + 3] = (uImmValue >> 48) & UINT8_C(0xff);
-                case 6: pThis->abInstr[off + 3] = (uImmValue >> 40) & UINT8_C(0xff);
-                case 5: pThis->abInstr[off + 3] = (uImmValue >> 32) & UINT8_C(0xff);
-                case 4: pThis->abInstr[off + 3] = (uImmValue >> 24) & UINT8_C(0xff);
-                case 3: pThis->abInstr[off + 2] = (uImmValue >> 16) & UINT8_C(0xff);
-                case 2: pThis->abInstr[off + 1] = (uImmValue >>  8) & UINT8_C(0xff);
+                case 8: pThis->abInstr[off + 3] = (uImmValue >> 56) & UINT8_C(0xff); /* fall thru */
+                case 7: pThis->abInstr[off + 3] = (uImmValue >> 48) & UINT8_C(0xff); /* fall thru */
+                case 6: pThis->abInstr[off + 3] = (uImmValue >> 40) & UINT8_C(0xff); /* fall thru */
+                case 5: pThis->abInstr[off + 3] = (uImmValue >> 32) & UINT8_C(0xff); /* fall thru */
+                case 4: pThis->abInstr[off + 3] = (uImmValue >> 24) & UINT8_C(0xff); /* fall thru */
+                case 3: pThis->abInstr[off + 2] = (uImmValue >> 16) & UINT8_C(0xff); /* fall thru */
+                case 2: pThis->abInstr[off + 1] = (uImmValue >>  8) & UINT8_C(0xff); /* fall thru */
                 case 1: pThis->abInstr[off] = uImmValue & UINT8_C(0xff);
                     break;