[9] RFR(S): 8143081: [ctw] Test CompileTheWorld.java needs to be updated for Jigsaw

Igor Ignatyev igor.ignatyev at oracle.com
Fri Jun 24 17:03:41 UTC 2016


Hi Tatiana,

1st of all, thanks for fixing that.

I’ve one suggest: could you please add a comment why we need to call System::exit and store cout/cerr in CompileTheWorld.java? otherwise looks good to me.

Thanks,
— Igor

> On Jun 23, 2016, at 8:19 PM, Tatiana Pivovarova <tatiana.pivovarova at oracle.com> wrote:
> 
> Hi guys!
> 
> Please review this patch
> 
> To run CompileTheWorld harness with new jdk9 it is need to add compatibility to read .jimage files and to remove hardcoded links to rt.jar and "sun.boot.class.path" property.
> 
> bug: https://bugs.openjdk.java.net/browse/JDK-8143081
> webrev: http://cr.openjdk.java.net/~tpivovarova/8143081/webrev.01/
> 
> Tested locally by compilation lib/modules.
> 
> Thanks,
> Tatiana
> 



More information about the hotspot-compiler-dev mailing list