Hi Shura,

test/java/lang/Class/GetModuleTest.java
   This can use java.base/jdk.internal.misc.Unsafe instead.

Other @modules looks okay.

Regarding the executeTestJava change, if you drop the VM options, this test 
will only exercise the default launcher setting.  The hotspot nightlies depend 
on VM options to exercise different configurations.  I think many tests that 
launch a modular test will run into the same issue.  It’s not ideal to drop the 
VM options then.  One alternative is to have a new 
ProcessTool::executeModularTest that will take test.vm.opts and test.java.opts 
but dropping -limitmods, if any.

Mandy

Reply via email to