RFR: 8308850: Change JVM options with small ranges from 64 to 32 bits, for globals.hpp [v6]
Coleen Phillimore
coleenp at openjdk.org
Thu Aug 10 02:06:38 UTC 2023
On Wed, 9 Aug 2023 14:54:04 GMT, Afshin Zafari <azafari at openjdk.org> wrote:
>> The JVM options that are in range of `int` type are converted in `globals.hpp` and other files where affected.
>>
>> ### Tests
>> tiers 1-4 passed for linux-x64, linux-x64-debug, windows-x64, windows-x64-debug
>> tier1 all passed.
>
> Afshin Zafari has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains six commits:
>
> - Merge branch 'master' into wconv_jvm_options
> - Matched with master.
> - checked all usages of changed options.
> - backslash aligned
> - Review comments applied.
> - Fixed intx->int for Wconversion warnings.
The criteria for choosing flags was whether they were causing -Wconversion warnings, which this one doesn't. That said, it would be consistent to change this one since other AllocatePrefetch flags are changed. There are likely many other flags that can be int not intx. They can be changed in a different patch.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/15164#issuecomment-1672419781
More information about the hotspot-dev
mailing list