RFR: 8359227: Code cache/heap size options should be size_t [v3]

Kim Barrett kbarrett at openjdk.org
Wed Jun 18 17:05:49 UTC 2025


On Wed, 18 Jun 2025 15:52:09 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 with a new target base due to a merge or a rebase. The pull request now contains 11 commits:
> 
>  - Merge branch 'master' into code-cache-sizes
>  - update copyrights
>  - remove leftover <type_traits> include
>  - fix whitebox access to code cache size configs
>  - VMPageSizeConstraintFunc
>  - CodeCacheMinBlockLength
>  - CodeCacheExpansionSize
>  - various CodeHeapSize options
>  - CodeCacheMinimumUseSpace
>  - Initial/ReservedCodeCacheSize
>  - ... and 1 more: https://git.openjdk.org/jdk/compare/ba32b78b...78f1f7b9

Sorry to request re-review, but there was a merge conflict, which I fixed, and
then another merge conflict appeared while I was testing that one.  Sigh :(

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

PR Comment: https://git.openjdk.org/jdk/pull/25791#issuecomment-2985056595


More information about the graal-dev mailing list