RFR: 8338021: Support new unsigned and saturating vector operators in VectorAPI [v32]

Emanuel Peter epeter at openjdk.org
Thu Oct 24 14:07:24 UTC 2024


On Thu, 24 Oct 2024 13:59:58 GMT, Jatin Bhateja <jbhateja at openjdk.org> wrote:

>> src/hotspot/cpu/x86/x86.ad line 10593:
>> 
>>> 10591:   match(Set dst (SaturatingAddV src1 src2));
>>> 10592:   match(Set dst (SaturatingSubV src1 src2));
>>> 10593:   format %{ "vector_addsub_saturating_subword $dst, $src1, $src2" %}
>> 
>> Could the `Opcode` be put into the `format` string? Not strictly necessary, but would be neat.
>
> Desirable future extension, but its not related to this specific PR.

Well, here it would be especially interesting, because it would tell us if we have a `sub` or an `add`.

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

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


More information about the hotspot-compiler-dev mailing list