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 needed, but they will be handled in separate bugs and code reviews.

can't really comment but does sjavac really need to mess with these:

! ADD_JVM_ARG_IF_OK([-XX:PermSize=32m],SJAVAC_SERVER_JAVA,[$SJAVAC_SERVER_JAVA]) ! ADD_JVM_ARG_IF_OK([-XX:MaxPermSize=160m],SJAVAC_SERVER_JAVA,[$SJAVAC_SERVER_JAVA]) ! ADD_JVM_ARG_IF_OK([-XX:ThreadStackSize=$STACK_SIZE],SJAVAC_SERVER_JAVA,[$SJAVAC_SERVER_JAVA])

??

David

/Erik

Reply via email to