RFR: 8342676: Unsigned Vector Min / Max transforms [v6]
Emanuel Peter
epeter at openjdk.org
Thu Apr 17 09:08:48 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
The code looks good to me. Let me run some more testing. Please ping me again in 24h+ and I can approve after the Easter weekend :)
-------------
PR Review: https://git.openjdk.org/jdk/pull/21604#pullrequestreview-2775089008
More information about the hotspot-compiler-dev
mailing list