On Thu, 15 Apr 2021 13:56:12 GMT, Alexander Scherbatiy <alex...@openjdk.org> 
wrote:

>> This is a proposal for cross compiling JavaFX base modules (excluding media 
>> and webkit) for Windows AArch64 (ARM64).
>> 
>> Main changes:
>> - prismES2 native compilation is moved under IS_INCLUDE_ES2 condition
>> - HOST_ARCH and TARGET_ARCH are retrieved from  ext.OS_ARCH and 
>> ext.TARGET_ARCH using substitution aarch64 to arm64 and amd64 to x64
>> - VCARCH is set to  "${HOST_ARCH}_${TARGET_ARCH}" architecture for cross 
>> compilation. VCARCH is set to x64 for amd64 target architecture (according 
>> to the [vcvarsall.bat 
>> doc](https://docs.microsoft.com/en-us/cpp/build/building-on-the-command-line?view=msvc-160#developer_command_file_locations)
>>  x64 and amd64 are interchangeable)
>> - arm64 rc.exe and fxc.exe execution fails on non arm64 host. The fix checks 
>> that and falls back to host rc.exe and fxc.exe. The right way would be to 
>> use rc.exe and fxc.exe from arm64 but I did not find a right way to run them 
>> on host system.
>> 
>> I also looked which changes are required to build media module for Windows 
>> aarch64.
>> The main changes could be using:
>> - `ARCH=arm64` for building media libs in build.gradle file
>> - `-MACHINE:arm64` LDFLAGS in media make files
>> - `msvc_build/aarch64/aarch64_include` header files for include, 
>> `src/aarch64/win64_armasm.S` for ASM_SOURCES, `armasm64.exe` for ML in 
>> gstreamer/projects/win/glib-lite/Makefile.glib file and corresponding 
>> include in gstreamer/projects/win/glib-lite/Makefile.gobject file
>> - setting `ENABLE_SIMD_SSE2` to 0 in ColorConverter.c in the similar way how 
>> it is done for Apple Silicon port
>> 
>> In this way the media is build but it is crashed when I run a JavaFX sample 
>> with video.
>> Is it possible to send the media Windows aarch64 port to review and 
>> investigate the crash in the separate fix?
>
> Alexander Scherbatiy has updated the pull request incrementally with one 
> additional commit since the last revision:
> 
>   Add CONVERTED_OS_ARCH and CONVERTED_TARGET_ARCH variables

build.gradle line 264:

> 262:         case "amd64" : return "x64"
> 263:         default: return arch
> 264:     }

I think a function that gets the "converted" arch is very useful, but I'm not 
sure what the resulting value should be. In case of arm64/aarch64, I have s 
slight preference for aarch64, but I'm not against arm64.

-------------

PR: https://git.openjdk.java.net/jfx/pull/439

Reply via email to