RFR: 8281455: Change JVM options with small ranges from 64 to 32 bits, for gc_globals.hpp [v2]
Kim Barrett
kbarrett at openjdk.org
Wed Jun 7 15:38:56 UTC 2023
On Wed, 7 Jun 2023 11:29:06 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:
>
> some more intx to int conversions.
There are a lot of uses of the options that have casts, many of which ought to be removable with this change.
Some of those might pull more tangled strings, but it would be nice if there was a plan to look at those too.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/14259#issuecomment-1581077477
More information about the hotspot-gc-dev
mailing list