RFR: 8308850: Change JVM options with small ranges from 64 to 32 bits, for globals.hpp [v2]

Afshin Zafari azafari at openjdk.org
Tue Aug 8 09:22:16 UTC 2023


On Mon, 7 Aug 2023 02:01:17 GMT, David Holmes <dholmes at openjdk.org> wrote:

>> src/hotspot/share/opto/macro.cpp line 1841:
>> 
>>> 1839:       // Generate several prefetch instructions.
>>> 1840:       int step_size = AllocatePrefetchStepSize;
>>> 1841:       int distance = AllocatePrefetchDistance;
>> 
>> I think this is a step in the wrong direction.  Shouldn't these remain uint?
>
> This change also doesn't fit the PR description. There is no need to change this in this PR (or even at all).

Okay. Reverted back.

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

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


More information about the graal-dev mailing list