On Mon, 2 Sep 2024 12:17:08 GMT, Jatin Bhateja <jbhat...@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
