RFR: 8333791: Fix memory barriers for @Stable fields [v2]

Christian Hagedorn chagedorn at openjdk.org
Mon Aug 5 14:06:34 UTC 2024


On Mon, 5 Aug 2024 08:09:02 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:

>> IIUC, you want to somehow have `-Xbootclasspath/a:path_to_your_ir_test`. I guess that's currently not so easy to do and probably needs some IR framework support.
>> 
>> I've had a quick look. How about something like that (just prototyped, not fully tested): [ir_framework.patch](https://github.com/user-attachments/files/16471225/ir_framework.log)
>> 
>> Then you could use it like that:
>> 
>>         TestFramework testFramework = new TestFramework();
>>         testFramework
>>                 .addFlags("...", "...")
>>                 .addTestClassesToBootClassPath()
>>                 .start();
>> 
>> 
>> If that works for all your tests, then you could either add the IR framework changes directly to this patch or we do it in preceding RFE separately.
>
> Yeah, OK, let's do IR Framework update separately. I am planning to have this fix backported, so I would like to have test infra fixes also be more or less cleanly backportable. @chhagedorn, are you taking point on this, or should I?

Sounds good, I can take care of it tomorrow. I will ping you in the PR once it is out.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/19635#discussion_r1704174548


More information about the hotspot-dev mailing list