Hello Vladimir,

These changes look innocent enough to me. They aren't actually adding macosx-aarch64 support, they are just removing two minor (and more likely OS version related) hurdles from the build. You still have to provide the actual configuration on the configure command line as is shown in your example. Before we can call build system support, we would need configure to automatically setup those flags and add a separate parameter for the JNF framework. So, given that, I don't think this change warrants a JEP in itself.

My only complaint is that you revert jib-profiles.js. That file is only used internally at Oracle. If/when we need it to support macosx-aarch64, we will provide those changes.

I must say I'm happy to see you managed to get a working build configuration with just this though!

/Erik

On 2020-08-01 00:24, Vladimir Kempik wrote:
Hello

Please review this change for JDK-8250876

This changeset adds support for macos/aarch64 into build system.
It will allow to crosscompile for macos/aarch64 using intel mac as well.

This changeset does NOT address some arm specific issues in the macos related 
code, we plan to do that in s separate commit.

An example of configure to cross-compile for macos/arm64:

--with-boot-jdk=/path/to/java/ --with-build-jdk=/path/to/same/java/as/compiled  
--disable-warnings-as-errors --with-jvm-variants=zero 
--openjdk-target=aarch64-apple-darwin --with-extra-cflags='-arch arm64' 
--with-extra-ldflags='-arch arm64 -F/Path/To/Folder/Containing/JNF_framework/' 
—with-extra-cxxflags='-arch arm64’

JNF.framework is missing arm64 part as of next macos release, but Apple has 
opensourced it.

Fix to adlc were needed due to it using symbols from stdc++ and not linking to 
it, so it fails when doing make images.

The webrev: http://cr.openjdk.java.net/~vkempik/8250876/webrev.00/
The bug: https://bugs.openjdk.java.net/browse/JDK-8250876

Testing: jdk/submit.

Thanks, Vladimir.

Reply via email to