On 06/12/2014 01:36, Kumar Srinivasan wrote:
Alan,

The heap needs to be increased (1.2GB) for this test, I think the default provided
by the launcher ergonomics is not sufficient on certain platforms. I've
tested the patch with JPRT.

Thanks
Kumar


diff --git a/test/tools/pack200/Pack200Props.java b/test/tools/pack200/Pack200Props.java
--- a/test/tools/pack200/Pack200Props.java
+++ b/test/tools/pack200/Pack200Props.java
@@ -62,6 +62,7 @@

         List<String> cmdsList = new ArrayList<>();
         cmdsList.add(Utils.getPack200Cmd());
+        cmdsList.add("-J-Xmx1280m");
         cmdsList.add("--effort=1");
         cmdsList.add("--verbose");
         cmdsList.add("--no-gzip");
This looks okay to me although 1.2GB makes me wonder if you might still need -Xshare:off on Windows 32-bit.

-Alan

Reply via email to