RFR: 8347459: C2: missing transformation for chain of shifts/multiplications by constants [v9]

Marc Chevalier duke at openjdk.org
Tue Mar 11 09:38:10 UTC 2025


On Mon, 10 Mar 2025 15:26:07 GMT, Emanuel Peter <epeter at openjdk.org> wrote:

>> Marc Chevalier has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Random testing, trying...
>
> src/hotspot/share/opto/memnode.cpp line 3536:
> 
>> 3534: // It is thus useful to handle the case where conIL > conIR.
>> 3535: //
>> 3536: // Let's assume we have the following 32 bits integer that we want to stuff in 8 bits char:
> 
> `char` actually is `16` bits unsigned in Java ;)

Yes, right. I replaced with byte.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/23728#discussion_r1988803887


More information about the hotspot-compiler-dev mailing list