RFR: 8323220: Reassociate loop invariants involved in Cmps and Add/Subs [v14]

Tue Apr 9 15:56:15 UTC 2024

On Mon, 25 Mar 2024 06:19:42 GMT, Emanuel Peter <epeter at openjdk.org> wrote:

>> Joshua Cao 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 17 additional commits since the last revision:
>> 
>>  - Merge branch 'master' into licm
>>  - @run driver -> @run main
>>  - Add tests for add/sub reassociation
>>  - Merge branch 'master' into licm
>>  - Make inputs deterministic. Make size an arg. Fix comments. Formatting.
>>  - Update test to utilize @setup method for arguments
>>  - Merge branch 'master' into licm
>>  - Add correctness test for some random tests with random inputs
>>  - Add some correctness tests where we do reassociate
>>  - Remove unused TestInfo parameter. Have some tests exit mid-loop.
>>  - ... and 7 more: https://git.openjdk.org/jdk/compare/1a3ba540...32cb9c0d
>
> Code looks good, running testing now... Ping me again in 2 days if I don't report back by then ;)

@eme64 could you take another pass at this?

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

PR Comment: https://git.openjdk.org/jdk/pull/17375#issuecomment-2045539108