RFR: 8308850: Change JVM options with small ranges from 64 to 32 bits, for globals.hpp
Coleen Phillimore
coleenp at openjdk.org
Mon Aug 7 14:46:31 UTC 2023
On Mon, 7 Aug 2023 12:02:14 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:
>> The only reason to use `intx` is if this should be 32-bit on 32-bit and 64-bit on 64-bit. I don't think that is the case here.
>
> The reason that this should be reverted is that to fix it, you have to fix all the platform code and that seems out of scope for fixing Wconversion warnings.
Actually this counts bytecodes, so it could easily become 64 bit on 64 bit platforms.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/15164#discussion_r1285976633
More information about the graal-dev
mailing list