stan/jdk-24
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

8342683: Use non-short forward jump when passing stop()

Browse Source 
Reviewed-by: aboldtch, shade, kvn
This commit is contained in:
Fredrik Bredberg 2024-10-23 15:26:32 +00:00
parent 964d8d2234
commit afb62f7349
1 changed files with 5 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
src/hotspot/cpu/x86/c2_MacroAssembler_x86.cpp
View File

@ -837,8 +837,9 @@ void C2_MacroAssembler::fast_unlock_lightweight(Register obj, Register reg_rax,
#ifdef ASSERT #ifdef ASSERT
// Check that unlocked label is reached with ZF set. // Check that unlocked label is reached with ZF set.
Label zf_correct; Label zf_correct;
Label zf_bad_zero;
jcc(Assembler::zero, zf_correct); jcc(Assembler::zero, zf_correct);
stop("Fast Unlock ZF != 1"); jmp(zf_bad_zero);
#endif #endif
bind(slow_path); bind(slow_path);
@ -847,8 +848,10 @@ void C2_MacroAssembler::fast_unlock_lightweight(Register obj, Register reg_rax,
} }
#ifdef ASSERT #ifdef ASSERT
// Check that stub->continuation() label is reached with ZF not set. // Check that stub->continuation() label is reached with ZF not set.
jccb(Assembler::notZero, zf_correct); jcc(Assembler::notZero, zf_correct);
stop("Fast Unlock ZF != 0"); stop("Fast Unlock ZF != 0");
bind(zf_bad_zero);
stop("Fast Unlock ZF != 1");
bind(zf_correct); bind(zf_correct);
#endif #endif
// C2 uses the value of ZF to determine the continuation. // C2 uses the value of ZF to determine the continuation.