On 2018-05-07 08:40, Erik Joelsson wrote:
Looks good to me.

Note for future, I would like it even more if we got rid of the pre-defined jvm.cfg altogether and just always generated it. The build itself shouldn't be dictating artificial limitations on build parameters. If we want to enforce limitations those should be explicit instead.

Just to clarify, I mean implicit artificial limitations.

/Erik
/Erik


On 2018-05-05 04:26, Aleksey Shipilev wrote:
RFE:
   https://bugs.openjdk.java.net/browse/JDK-8202683

Fix:
   http://cr.openjdk.java.net/~shade/8202683/webrev.01/

Minimal VM is targeted to 32-bit only, but hear me out. Recent build system changes, notably conditional GC compilation, requires build/testing with some GCs disabled. Ultimately, we want minimal VM to be buildable at all times. Today, we need to cross-compile to x86_32 to verify that, but with this change, we can build minimal on x86_64, thus simplifying our day-to-day jobs. The change itself generates jvm.cfg for minimal VM properly on both bitnesses.

Testing: x86_64 server/minimal builds, x86_32 server/minimal builds

Thanks,
-Aleksey



Reply via email to