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 hotspot-dev
mailing list