RFR: 8377172: Change datatype of CodeEntryAlignment to uint [v4]
Leo Korinth
lkorinth at openjdk.org
Tue Feb 10 14:08:44 UTC 2026
> The main idea of this change is to remove an implicitly narrowing conversion to int from intx (`int code_alignment()`).
>
> CodeEntryAlignment is constrained to not be bigger than CodeCacheSegmentSize that is itself constrained to be in the range(1, 1024). The data type should really be a uint16_t as its allowed range is [16-1024], but we have no such support for VM flags. uint will do, and will simplify the code greatly (five less casts, six if we count the implicit narrowing cast).
>
> This change is inspired by my change 8376892: Allow conversion warnings in subsets of the code base.
>
> Completed test tier 1-3.
Leo Korinth has updated the pull request incrementally with one additional commit since the last revision:
copyright
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/29637/files
- new: https://git.openjdk.org/jdk/pull/29637/files/0277648c..00bf1b01
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=29637&range=03
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=29637&range=02-03
Stats: 14 lines in 14 files changed: 0 ins; 0 del; 14 mod
Patch: https://git.openjdk.org/jdk/pull/29637.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/29637/head:pull/29637
PR: https://git.openjdk.org/jdk/pull/29637
More information about the hotspot-dev
mailing list