RFR: 8359227: Code cache/heap size options should be size_t [v2]
Manuel Hässig
mhaessig at openjdk.org
Wed Jun 18 10:36:29 UTC 2025
On Fri, 13 Jun 2025 18:46:58 GMT, Kim Barrett <kbarrett at openjdk.org> wrote:
>> Please review this change that makes the various code cache/heap size options
>> consistently be of type size_t.
>>
>> The shared declarations for these options were all uintx. These options all
>> may have platform-defined values. Some of those platform-specific definitions
>> were uintx, some were size_t, and some were intx(!). This change makes them
>> all consistently size_t.
>>
>> More details in the first comment.
>>
>> Testing: mach5 tier1-6
>> GHA testing in-progress
>
> Kim Barrett has updated the pull request incrementally with two additional commits since the last revision:
>
> - update copyrights
> - remove leftover <type_traits> include
Thank you for addressing the comments. Looks good to me!
-------------
Marked as reviewed by mhaessig (Author).
PR Review: https://git.openjdk.org/jdk/pull/25791#pullrequestreview-2938683405
More information about the graal-dev
mailing list