RFR: 8281455: Change JVM options with small ranges from 64 to 32 bits, for gc_globals.hpp
Thomas Schatzl
tschatzl at openjdk.org
Wed Jun 7 08:57:08 UTC 2023
On Thu, 1 Jun 2023 11:15:48 GMT, Afshin Zafari <azafari at openjdk.org> wrote:
> The `uintx/intx` options whose ranges are small enough are changed to `uint/int`, otherwise gcc complains
> when `-Wconversion` is used in build.
> Their uses in printf formats are changed accordingly.
>
> ### Tests
> Locally hotspot:tier1 tested on linux-x64
> mach5 tiers 1-4 on Linux and Windows 64bits platforms passed.
Although it's a bit early to review, when I looked at this briefly, I believe also the `uintx` that have a `juint_max` upper range can be replaced with `uint` (and use the equivalent `UINT_MAX` as upper range, `j`-types should only be used with JNI code).
-------------
PR Comment: https://git.openjdk.org/jdk/pull/14259#issuecomment-1573295241
More information about the hotspot-gc-dev
mailing list