RFR: 8138689 use package for /compiler/whitebox common classes

dmitrij pochepko dmitrij.pochepko at oracle.com
Tue Nov 3 21:41:52 UTC 2015


Hi,

a link to webrev: http://cr.openjdk.java.net/~dpochepk/8138689/webrev.01/

Thanks,
Dmitrij
> Hi,
>
> please review fix for https://bugs.openjdk.java.net/browse/JDK-8138689 
> - use package for /compiler/whitebox common classes
>
> Description:
>
> A common class CompilerWhiteBoxTest was moved to compiler.whitebox 
> package, so, respective changes were made in tests to reflect that.
> Also, a CompilerWhiteBoxTest has few protected static fields and 
> methods moved to public for tests to access it.
>
> Test changes: most of tests has 2 changes:
> a) changed @library tag entry from '/compiler/whitebox' to '/'
> b) added import compiler.whitebox.CompilerWhiteBoxTest;
>
> also compiler/whitebox tests which has CompilerCommand compileonly 
> directive were changed to have package name in compileonly parameter.
> another small change is in LevelTransitionTest isTrivial() detection 
> which can't access SimpleTestCase, so, testcase filtering is done by 
> testcase name now.
>
> Testing: I've tested it on linux-amd64
>
> Thanks,
> Dmitrij

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/attachments/20151104/e5031f9a/attachment-0001.html>


More information about the hotspot-compiler-dev mailing list