RFR: 8338021: Support saturating vector operators in VectorAPI [v4]

[Sandhya Viswanathan]

On Mon, 2 Sep 2024 12:17:08 GMT, Jatin Bhateja <jbhateja at openjdk.org> wrote:

>> src/hotspot/cpu/x86/x86.ad line 10656:
>> 
>>> 10654:   match(Set dst (SaturatingSubVI src1 src2));
>>> 10655:   match(Set dst (SaturatingSubVL src1 src2));
>>> 10656:   effect(TEMP ktmp);
>> 
>> This needs TEMP dst as well.
>
> There is no use of either of the source operands after assignment to dst in the macro assembly routine.

Sorry, I meant this for another instruct.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/20507#discussion_r1742767372