RFR(M): 8150054: CompilerControl: add Xmixed for execution

Vladimir Kozlov vladimir.kozlov at oracle.com
Tue Mar 15 20:02:10 UTC 2016

Looks good


On 3/15/16 6:44 AM, Nils Eliasson wrote:
> Hi,
> Please review this improvement of the compiler control tests. There are
> many files in the diff but only three kind of changes:
> 1) Most of the test spawn a separate VM that it can control with flags
> and diagnostic commands. The launching VM just need to setup the test
> environment and is not part of the test. We can therefore change "@run
> main/othervm" to " @run driver". A driver can not have any flags and are
> not affected by any flag rotation.
> 2) Change the "@run main ClassFileInstaller sun.hotspot.WhiteBox " to
> driver too. This could be done for all the ~400 test but I have limited
> myself to the compiler control test in this bug. In -Xcomp-batches this
> saves ~5 seconds per ClassFileInstaller invocation on a fast machine.
> 3) Add the -Xmixed flag to Scenario.java to prevent -Xcomp from
> interfering with testing printing and logging. It causes long run times
> and huge logs.
> These three changes together save more than 10 minutes of test time on a
> -Xcomp-batch on a fast x64 workstation.
> Bug: https://bugs.openjdk.java.net/browse/JDK-8150054
> Webrev: http://cr.openjdk.java.net/~neliasso/8150054/webrev.01
> Regards,
> Nils

More information about the hotspot-compiler-dev mailing list