RFR: 8304915: Create jdk.internal.util.Architecture enum and apply [v4]

Roger Riggs rriggs at openjdk.org
Wed Apr 5 20:43:27 UTC 2023


On Wed, 5 Apr 2023 20:25:43 GMT, Bernd <duke at openjdk.org> wrote:

>> Roger Riggs has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Correct spelling of isAARCH64 in WIndows AttachProviderImpl
>
> src/java.base/share/classes/jdk/internal/foreign/CABI.java line 48:
> 
>> 46:         // might be running in a 32-bit VM on a 64-bit platform.
>> 47:         // addressSize will be correctly 32
>> 48:         if (Architecture.isX64() && ADDRESS_SIZE == 64) {
> 
> Is there a difference to Architecture.is64bit (I.e. is the later or the former runtime vs compiletime

There should be no difference; I was hesitant to drop the ADDRESS_SIZE check without knowing more about the foreign api dependencies.  ADDRESS_SIZE is computed (I think) from `UNSAFE.ADDRESS_SIZE * 8`.
But I can't think of how it can be different than the CPU_BITS that are defined when the JDK is built.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/13357#discussion_r1159001239



More information about the client-libs-dev mailing list