RFR: 8342676: Unsigned Vector Min / Max transforms [v6]
Emanuel Peter
epeter at openjdk.org
Mon Apr 21 10:24:42 UTC 2025
On Mon, 14 Apr 2025 13:50:17 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 incrementally with one additional commit since the last revision:
>
> Comment refinement
Testing looks good, code too 😊
You'll need a second review though :)
-------------
Marked as reviewed by epeter (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/21604#pullrequestreview-2780983729
More information about the hotspot-compiler-dev
mailing list