Re:  RFR(S): 8143081: [ctw] Test CompileTheWorld.java needs to be updated for Jigsaw
igor.ignatyev at oracle.com
Mon Jun 27 19:39:38 UTC 2016
Thank you for adding comments. Looks good to me.
----- Reply message -----
From: "Tatiana Pivovarova" <tatiana.pivovarova at oracle.com>
To: "Igor Ignatyev" <igor.ignatyev at oracle.com>
Cc: <hotspot-compiler-dev at openjdk.java.net>
Subject:  RFR(S): 8143081: [ctw] Test CompileTheWorld.java needs to be updated for Jigsaw
Date: Mon, Jun 27, 2016 16:32
Thanks for your review. I added these comments.
webrev : http://cr.openjdk.java.net/~tpivovarova/8143081/webrev.04/
On 06/24/2016 08:03 PM, Igor Ignatyev
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.
On Jun 23, 2016, at 8:19 PM, Tatiana Pivovarova <tatiana.pivovarova at oracle.com> wrote:
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.
Tested locally by compilation lib/modules.
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the hotspot-compiler-dev