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

Wed Jan 5 12:28:26 UTC 2022

On Tue, 4 Jan 2022 13:48:48 GMT, Christian Hagedorn <chagedorn at openjdk.org> wrote:

>> 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
>
> src/hotspot/share/opto/loopopts.cpp line 377:
> 
>> 375:     Node* add_ctrl = get_ctrl(add);
>> 376:     IdealLoopTree *add_loop = get_loop(add_ctrl);
>> 377:     //assert( n_loop == add_loop, "" );
> 
> Probably a left-over from the bailout below. Could be removed.

You probably missed that one but it's just a minor thing.

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

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