RFR: 8349361: C2: RShiftL should support all applicable transformations that RShiftI does [v7]

Jasmine Karthikeyan jkarthikeyan at openjdk.org
Sun Feb 9 06:06:09 UTC 2025


On Fri, 7 Feb 2025 09:52:51 GMT, Roland Westrelin <roland at openjdk.org> wrote:

>> This change refactors `RShiftI`/`RshiftL` `Ideal`, `Identity` and
>> `Value` because the `int` and `long` versions are very similar and so
>> there's no logic duplication. In the process, support for some extra
>> transformations is added to `RShiftL`. I also added some new test
>> cases.
>
> Roland Westrelin has updated the pull request incrementally with one additional commit since the last revision:
> 
>   review

Thanks for the update, it looks good!

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

Marked as reviewed by jkarthikeyan (Committer).

PR Review: https://git.openjdk.org/jdk/pull/23438#pullrequestreview-2604126801


More information about the hotspot-compiler-dev mailing list