Re: RFR: 8014514: Fix jvm args for sjavac

2013-05-15 Thread Erik Joelsson
On 2013-05-15 06:21, David Holmes wrote: On 15/05/2013 12:34 AM, Erik Joelsson wrote: This patch tries to be smarter when figuring out optimal jvm args for sjavac, especially if the jvm used is 32bit. http://cr.openjdk.java.net/~erikj/8014514/webrev.root.01/ For this to work properly, two

Re: RFR: 8014514: Fix jvm args for sjavac

2013-05-15 Thread Erik Joelsson
On 2013-05-15 09:01, Erik Joelsson wrote: On 2013-05-15 06:21, David Holmes wrote: On 15/05/2013 12:34 AM, Erik Joelsson wrote: This patch tries to be smarter when figuring out optimal jvm args for sjavac, especially if the jvm used is 32bit.

Re: RFR: 8014514: Fix jvm args for sjavac

2013-05-14 Thread Tim Bell
Erik: This patch tries to be smarter when figuring out optimal jvm args for sjavac, especially if the jvm used is 32bit. http://cr.openjdk.java.net/~erikj/8014514/webrev.root.01/ Looks good. Tim

Re: RFR: 8014514: Fix jvm args for sjavac

2013-05-14 Thread David Holmes
On 15/05/2013 12:34 AM, Erik Joelsson wrote: This patch tries to be smarter when figuring out optimal jvm args for sjavac, especially if the jvm used is 32bit. http://cr.openjdk.java.net/~erikj/8014514/webrev.root.01/ For this to work properly, two patches to the sjavac src langtools are also