RFR: 8342676: Unsigned Vector Min / Max transforms [v2]

[Xiaohong Gong]

On Tue, 7 Jan 2025 08:58:12 GMT, Jatin Bhateja <jbhateja at openjdk.org> wrote:

>> Adding following IR transforms for unsigned vector Min / Max nodes.
>> 
>> => UMinV (UMinV(a, b), UMaxV(a, b)) => UMinV(a, b)
>> => UMinV (UMinV(a, b), UMaxV(b, a)) => UMinV(a, b)
>> => UMaxV (UMinV(a, b), UMaxV(a, b)) => UMaxV(a, b)
>> => UMaxV (UMinV(a, b), UMaxV(b, a)) => UMaxV(a, b)
>> => UMaxV (a, a) => a
>> => UMinV (a, a) => a
>> 
>> New IR validation test accompanies the patch.
>> 
>> This is a follow-up PR for https://github.com/openjdk/jdk/pull/20507
>> 
>> Best Regards,
>> Jatin
>
> Jatin Bhateja 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 five additional commits since the last revision:
> 
>  - Updating copyright year of modified files
>  - Merge branch 'master' of http://github.com/openjdk/jdk into JDK-8342676
>  - Update IR transforms and tests
>  - Merge branch 'master' of http://github.com/openjdk/jdk into JDK-8342676
>  - 8342676: Unsigned Vector Min / Max transforms

test/hotspot/jtreg/compiler/lib/ir_framework/IRNode.java line 1213:

> 1211: 
> 1212:    public static final String UMIN_VB = VECTOR_PREFIX + "UMIN_VB" + POSTFIX;
> 1213:     static {

Suggestion:

    public static final String UMIN_VB = VECTOR_PREFIX + "UMIN_VB" + POSTFIX;
    static {

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

PR Review Comment: https://git.openjdk.org/jdk/pull/21604#discussion_r1914397192