RFR: 8281455: Change JVM options with small ranges from 64 to 32 bits, for gc_globals.hpp [v4]

Coleen Phillimore coleenp at openjdk.org
Tue Jun 27 15:49:08 UTC 2023


On Mon, 26 Jun 2023 11:46:21 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.
>
> Afshin Zafari has updated the pull request incrementally with one additional commit since the last revision:
> 
>   review comments applied.

This looks good and definitely a step forward.

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

Marked as reviewed by coleenp (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/14259#pullrequestreview-1501297079


More information about the hotspot-gc-dev mailing list