RFR: 8304006: jlink should create the jimage file in the native endian for the target platform [v12]
Jaikiran Pai
jpai at openjdk.org
Fri Mar 24 02:18:45 UTC 2023
On Thu, 23 Mar 2023 18:30:40 GMT, Mandy Chung <mchung at openjdk.org> wrote:
>> Jaikiran Pai has updated the pull request incrementally with one additional commit since the last revision:
>>
>> ARM is 32 bit as per platform.m4 in OpenJDK build
>
> src/jdk.jlink/share/classes/jdk/tools/jlink/internal/JlinkTask.java line 823:
>
>> 821: if (order != null) {
>> 822: this.order = order;
>> 823: String targetPlatformVal = readJavaBaseTargetPlatform(cf);
>
> Thanks for the update.
>
> This should also do the fast-path check `isJavaBaseFromCurrentPlatform(javaBasePath)` as in line 838-860. I suggest to refactor line 838-860 to return the target platform (either runtime platform or from java.base in the given module path).
>
> I also suggest to include `endianness` in the `Platform` record class.
>
> I sent you a patch offline with these suggestions.
Thank you Mandy for the review and the improved patch. I've applied it and tests continue to pass. I've updated the PR with these changes.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/11943#discussion_r1147050227
More information about the core-libs-dev
mailing list