RFR(XS) : 8159149 : Several compiler tests fail with minimal VM

[Paul Sandoz]

> On 16 Jun 2016, at 06:27, Igor Ignatyev <igor.ignatyev at oracle.com> wrote:
> 
> http://cr.openjdk.java.net/~iignatyev/8159149/webrev.00/
>> 70 lines changed: 4 ins; 1 del; 65 mod;
> 
> Hi all,
> 
> could you please review the fix for a couple of compiler tests to make it possible to run them on minimal VM or exclude them if they are not applicable?
> 
> minimal VM doesn’t have several features such as jcmd, instrumentation agents and support only which is in java.compact2 module.
> - compiler/c2/8004867/TestIntUnsafeOrdered.java and compiler/codegen/8011901/Test8011901.java used sun.misc.Unsafe which is in jdk.unsupported module which is not in java.compact2 set, they tests were rewritten to use jdk.internal.misc.Unsafe instead.
> - compiler/jsr292/NonInlinedCall/RedefineTest.java uses instrumentation agent, compiler/compilercontrol/jcmd/PrintDirectivesTest.java uses jcmd, these features are not supported in minimal VM, the corresponding @requiers was added to exclude the tests
> - compiler/compilercontrol/mixed/RandomValidCommandsTest.java and compiler/compilercontrol/logcompilation/LogTest.java have indirect dependency on java.management module. this dependency is in driver code, not in test code, so converting ‘@run main’ to '@run driver’ makes it possible to run these tests on minimal VM. PS there is a plan to get rid of this dependency
> 
> JBS: https://bugs.openjdk.java.net/browse/JDK-8159149
> webrev: http://cr.openjdk.java.net/~iignatyev/8159149/webrev.00/
> testing: locally on macos w/ and w/o ‘-limitmods java.compact2’ and remotely on linux-x86 w/ ‘-minimal -limitmods java.compact2’
> 

+1 (mostly on on changes on using internal Unsafe), but the other changes look ok to me too.

Paul.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 841 bytes
Desc: Message signed with OpenPGP using GPGMail
URL: <http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/attachments/20160616/0704078d/signature-0001.asc>