RFR: 8364235: Fix for JDK-8361447 breaks the alignment requirements for GuardedMemory [v2]
David Holmes
dholmes at openjdk.org
Wed Jul 30 06:47:56 UTC 2025
On Tue, 29 Jul 2025 08:30:42 GMT, Axel Boldt-Christmas <aboldtch at openjdk.org> wrote:
>> David Holmes has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Correct comments as per reviewer comments
>
> src/hotspot/share/memory/guardedMemory.hpp line 45:
>
>> 43: * |------------------------------------------------------------
>> 44: * |base_addr | 0xABABABABABABABAB | Head guard |
>> 45: * |+16 | padding | For alignment |
>
> Is `+sizeof(Guard)` or `+GUARD_SIZE` an alternative to `+16`. 16 is a little out of the blue for me.
Fixed
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/26524#discussion_r2241688266
More information about the hotspot-runtime-dev
mailing list