RFR: 8273108: RunThese24H crashes with SEGV in markWord::displaced_mark_helper() after JDK-8268276

Sandhya Viswanathan sviswanathan at openjdk.java.net
Fri Dec 10 00:00:17 UTC 2021


On Thu, 9 Dec 2021 23:50:52 GMT, Scott Gibbons <duke at openjdk.java.net> wrote:

>> src/hotspot/cpu/x86/stubGenerator_x86_64.cpp line 6264:
>> 
>>> 6262:       __ jcc(Assembler::lessEqual, L_finalBit);
>>> 6263: 
>>> 6264:       __ mov64(rax, 0x0000ffffffffffff);
>> 
>> The constant should have an l suffix.
>
> I do not believe this is necessary.  There are multiple occurrences of mov64()s without the `l` suffix.  For example, lines 687-688:
> 
>     __ mov64(c_rarg3, 0x8000000000000000);
>     __ mov64(rax, 0x7fffffffffffffff);

You are right, the code looks good.

-------------

PR: https://git.openjdk.java.net/jdk/pull/6786


More information about the hotspot-compiler-dev mailing list