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

Kim Barrett kbarrett at openjdk.org
Wed Jun 18 15:52:09 UTC 2025


> 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

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

Changes: https://git.openjdk.org/jdk/pull/25791/files
  Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=25791&range=02
  Stats: 190 lines in 40 files changed: 1 ins; 0 del; 189 mod
  Patch: https://git.openjdk.org/jdk/pull/25791.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/25791/head:pull/25791

PR: https://git.openjdk.org/jdk/pull/25791


More information about the graal-dev mailing list