# HG changeset patch # User mdoerr # Date 1444082039 -7200 # Node ID 3c82f7ac03e6b1d7c175bab4eb8042a43944c073 # Parent 343abbc0657262fd9892a96aaadf8f5be279f44b 8138896: C1: NativeGeneralJump is mixed up with NativeCall in C1 patching code Reviewed-by: twisti, vlivanov diff -r 343abbc06572 -r 3c82f7ac03e6 hotspot/src/share/vm/c1/c1_LIRAssembler.cpp --- a/hotspot/src/share/vm/c1/c1_LIRAssembler.cpp Fri Nov 27 09:36:46 2015 +0100 +++ b/hotspot/src/share/vm/c1/c1_LIRAssembler.cpp Mon Oct 05 23:53:59 2015 +0200 @@ -36,7 +36,7 @@ // We must have enough patching space so that call can be inserted. // We cannot use fat nops here, since the concurrent code rewrite may transiently // create the illegal instruction sequence. - while ((intx) _masm->pc() - (intx) patch->pc_start() < NativeCall::instruction_size) { + while ((intx) _masm->pc() - (intx) patch->pc_start() < NativeGeneralJump::instruction_size) { _masm->nop(); } patch->install(_masm, patch_code, obj, info); diff -r 343abbc06572 -r 3c82f7ac03e6 hotspot/src/share/vm/c1/c1_Runtime1.cpp --- a/hotspot/src/share/vm/c1/c1_Runtime1.cpp Fri Nov 27 09:36:46 2015 +0100 +++ b/hotspot/src/share/vm/c1/c1_Runtime1.cpp Mon Oct 05 23:53:59 2015 +0200 @@ -1163,7 +1163,7 @@ } #endif - for (int i = NativeCall::instruction_size; i < *byte_count; i++) { + for (int i = NativeGeneralJump::instruction_size; i < *byte_count; i++) { address ptr = copy_buff + i; int a_byte = (*ptr) & 0xFF; address dst = instr_pc + i;