RFR: 8308850: Change JVM options with small ranges that get -Wconversion warnings to 32 bits [v7]

Thomas Schatzl tschatzl at openjdk.org
Fri Aug 11 07:34:59 UTC 2023


On Thu, 10 Aug 2023 17:57:12 GMT, Ioi Lam <iklam at openjdk.org> wrote:

>> Looks like from the constraints, it's allowed to be max_intx on some platforms, so changing this would be a functional change.
>> 
>> JVMFlag::Error AllocatePrefetchInstrConstraintFunc(intx value, bool verbose) {
>>   intx max_value = max_intx;
>> #if defined(X86)
>>   max_value = 3;
>> #endif
>
> OK, that sounds reasonable.

Fwiw, only x86 uses `AllocatePrefetchInstr`. Other platforms never use it, so its value for those is meaningless.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/15164#discussion_r1290976345


More information about the hotspot-dev mailing list