--- old/src/share/vm/code/dependencies.cpp 2017-06-28 15:43:52.000000000 -0700 +++ new/src/share/vm/code/dependencies.cpp 2017-06-28 15:43:52.000000000 -0700 @@ -272,6 +272,8 @@ case exclusive_concrete_methods_2: swap = (x->ident() > x2->ident() && x->as_metadata()->as_method()->holder() != ctxk); break; + default: + break; } if (swap) { ciBaseObject* t = x; x = x2; x2 = t; } @@ -420,7 +422,7 @@ case 1: deps->sort(sort_dep_value_arg_1, 1); break; case 2: deps->sort(sort_dep_value_arg_2, 2); break; case 3: deps->sort(sort_dep_value_arg_3, 3); break; - default: ShouldNotReachHere(); + default: ShouldNotReachHere(); break; } } return; @@ -434,7 +436,7 @@ case 1: deps->sort(sort_dep_arg_1, 1); break; case 2: deps->sort(sort_dep_arg_2, 2); break; case 3: deps->sort(sort_dep_arg_3, 3); break; - default: ShouldNotReachHere(); + default: ShouldNotReachHere(); break; } } } @@ -466,8 +468,9 @@ case unique_concrete_method: case exclusive_concrete_methods_2: return x->as_metadata()->as_method()->holder(); + default: + return NULL; // let NULL be NULL } - return NULL; // let NULL be NULL } Klass* Dependencies::ctxk_encoded_as_null(DepType dept, Metadata* x) { @@ -480,8 +483,9 @@ case exclusive_concrete_methods_2: assert(x->is_method(), "sanity"); return ((Method*)x)->method_holder(); + default: + return NULL; // let NULL be NULL } - return NULL; // let NULL be NULL } void Dependencies::encode_content_bytes() { @@ -1459,7 +1463,7 @@ // overpass clause is false if k == NULL, implies return true if // answer depends on overpass clause. return ! ( m == NULL || m -> is_static() || m -> is_abstract() || - m->is_overpass() && k != NULL && k -> is_abstract() ); + (m->is_overpass() && k != NULL && k -> is_abstract()) ); } @@ -1864,6 +1868,8 @@ tty->print_cr(" context interface = %s", k->external_name()); } break; + default: + break; } } if (nsup + nint != 0) {