RFR: 8290892: C2: Intrinsify Reference.reachabilityFence [v2]

[Vladimir Ivanov]

On Tue, 20 May 2025 08:49:58 GMT, Tobias Hartmann <thartmann at openjdk.org> wrote:

>> Vladimir Ivanov has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   review feedback
>
> test/hotspot/jtreg/compiler/c2/TestReachabilityFence.java line 93:
> 
>> 91:                 return payload[id][offset];
>> 92:             } finally {
>> 93: //                Reference.reachabilityFence(this);
> 
> Drive-by comment: Is this intentionally disabled?

Good catch. Uncommented.

(FTR I wasn't able to reproduce the problem with on-heap backing storage. Choosing between removing the test case which doesn't provoke the problem and keeping it, I chose the latter.)

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

PR Review Comment: https://git.openjdk.org/jdk/pull/25315#discussion_r2103471176