--- old/src/share/vm/opto/stringopts.cpp 2016-07-11 22:46:55.381346338 +0900 +++ new/src/share/vm/opto/stringopts.cpp 2016-07-11 22:46:55.241346828 +0900 @@ -157,7 +157,7 @@ // Null check of the return of toString which can simply be skipped. if (b->_test._test == BoolTest::ne && v2->bottom_type() == TypePtr::NULL_PTR && - value->in(true_path)->Opcode() == Op_CastPP && + value->in(true_path)->Opcode() == Opcodes::Op_CastPP && value->in(true_path)->in(1) == v1 && v1->is_Proj() && is_SB_toString(v1->in(0))) { return v1; @@ -343,8 +343,8 @@ // creates so put it on the worklist to be collapsed later. for (SimpleDUIterator i(projs.catchall_catchproj); i.has_next(); i.next()) { Node *use = i.get(); - int opc = use->Opcode(); - if (opc == Op_CreateEx || opc == Op_Region) { + Opcodes opc = use->Opcode(); + if (opc == Opcodes::Op_CreateEx || opc == Opcodes::Op_Region) { _stringopts->record_dead_node(use); } } @@ -679,9 +679,9 @@ // analysis doesn't get unhappy. while (dead_worklist.size() > 0) { Node* use = dead_worklist.pop(); - int opc = use->Opcode(); + Opcodes opc = use->Opcode(); switch (opc) { - case Op_Region: { + case Opcodes::Op_Region: { uint i = 1; for (i = 1; i < use->req(); i++) { if (use->in(i) != C->top()) { @@ -699,8 +699,8 @@ } break; } - case Op_AddP: - case Op_CreateEx: { + case Opcodes::Op_AddP: + case Opcodes::Op_CreateEx: { // Recurisvely clean up references to CreateEx so EA doesn't // get unhappy about the partially collapsed graph. for (SimpleDUIterator i(use); i.has_next(); i.next()) { @@ -712,7 +712,7 @@ C->gvn_replace_by(use, C->top()); break; } - case Op_Phi: + case Opcodes::Op_Phi: if (use->in(0) == C->top()) { C->gvn_replace_by(use, C->top()); } @@ -1057,12 +1057,12 @@ // already checked this continue; } - int opc = use->Opcode(); - if (opc == Op_CmpP || opc == Op_Node) { + Opcodes opc = use->Opcode(); + if (opc == Opcodes::Op_CmpP || opc == Opcodes::Op_Node) { ctrl_path.push(use); continue; } - if (opc == Op_CastPP || opc == Op_CheckCastPP) { + if (opc == Opcodes::Op_CastPP || opc == Opcodes::Op_CheckCastPP) { for (SimpleDUIterator j(use); j.has_next(); j.next()) { worklist.push(j.get()); } @@ -1462,7 +1462,7 @@ if (!dcon || !dbyte) { // Destination is UTF16. Inflate src_array into dst_array. kit.sync_kit(ideal); - if (Matcher::match_rule_supported(Op_StrInflatedCopy)) { + if (Matcher::match_rule_supported(Opcodes::Op_StrInflatedCopy)) { // Use fast intrinsic Node* src = kit.array_element_address(src_array, kit.intcon(0), T_BYTE); Node* dst = kit.array_element_address(dst_array, start, T_BYTE); @@ -1963,7 +1963,7 @@ // of the initialization is committed to memory before any code publishes // a reference to the newly constructed object (see Parse::do_exits()). assert(AllocateNode::Ideal_allocation(result, _gvn) != NULL, "should be newly allocated"); - kit.insert_mem_bar(Op_MemBarRelease, result); + kit.insert_mem_bar(Opcodes::Op_MemBarRelease, result); } else { result = C->top(); }