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