RFR: 8014514: Fix jvm args for sjavac

David Holmes david.holmes at oracle.com
Wed May 15 04:21:02 UTC 2013


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



More information about the build-dev mailing list