src/share/vm/opto/type.cpp

@@ -267,26 +267,29 @@
       }
     case T_ILLEGAL:
       // Invalid ciConstant returned due to OutOfMemoryError in the CI
       assert(Compile::current()->env()->failing(), "otherwise should not see this");
       return NULL;
-  }
+    default:
   // Fall through to failure
   return NULL;
+  }
 }
 
 static ciConstant check_mismatched_access(ciConstant con, BasicType loadbt, bool is_unsigned) {
   BasicType conbt = con.basic_type();
   switch (conbt) {
     case T_BOOLEAN: conbt = T_BYTE;   break;
     case T_ARRAY:   conbt = T_OBJECT; break;
+    default:                          break;
   }
   switch (loadbt) {
     case T_BOOLEAN:   loadbt = T_BYTE;   break;
     case T_NARROWOOP: loadbt = T_OBJECT; break;
     case T_ARRAY:     loadbt = T_OBJECT; break;
     case T_ADDRESS:   loadbt = T_OBJECT; break;
+    default:                             break;
   }
   if (conbt == loadbt) {
     if (is_unsigned && conbt == T_BYTE) {
       // LoadB (T_BYTE) with a small mask (<=8-bit) is converted to LoadUB (T_BYTE).
       return ciConstant(T_INT, con.as_int() & 0xFF);

@@ -1046,14 +1049,15 @@
   case Memory:
   case Bottom:
   case FloatBot:
   case DoubleBot:
     return false;  // never a singleton, therefore never empty
-  }
 
+  default:
   ShouldNotReachHere();
   return false;
+  }
 }
 
 //------------------------------dump_stats-------------------------------------
 // Dump collected statistics to stderr
 #ifndef PRODUCT

@@ -4078,10 +4082,13 @@
     case T_NARROWKLASS:
     case T_CONFLICT:
     case T_ILLEGAL:
     case T_VOID:
       etype = T_BYTE;           // will produce conservatively high value
+      break;
+    default:
+      break;
     }
     cache = res = arrayOopDesc::max_array_length(etype);
   }
   return res;
 }