RFR: 8278784: C2: Refactor PhaseIdealLoop::remix_address_expressions() so it operates on longs [v2]

Christian Hagedorn chagedorn at openjdk.java.net
Wed Jan 5 12:28:22 UTC 2022


On Wed, 5 Jan 2022 11:51:55 GMT, Roland Westrelin <roland at openjdk.org> wrote:

>> The logic in PhaseIdealLoop::remix_address_expressions() that's
>> specific to int nodes apply equally to long nodes. This change
>> refactors that code so it applies to both integer types. This improves
>> performance on some of panama's micro-benchmarks.
>
> Roland Westrelin has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains four additional commits since the last revision:
> 
>  - Christian's review
>  - Merge branch 'master' into JDK-8278784
>  - test
>  - remix address

Thanks for doing the updates, looks good!

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

Marked as reviewed by chagedorn (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/6892


More information about the hotspot-compiler-dev mailing list