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