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

[Emanuel Peter]

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

>> Also: all of these cases load, and directly store again. Does that not mean all tests will probably pick the "..._mem" backend operations? Or do we actually end up testing all backend operations with the tests we have here?
>
> To exercise non memory operand pattern we need a vector operation padding layer after load vector, this will always ensure that selector pick all register operands flavor of instruction. Since its a generic limitation, do you think we should float it as a separate PR? 
> 
> I have create an RFE https://bugs.openjdk.org/browse/JDK-8342959  for reference. Given that we have moved IR tests out this PR on the grounds of review complexity, lets not add more code here.

Ok, we can file a separate RFE. Though I really have voiced 2 concerns:
- Making sure we always test `_mem` and `_reg` variants in the backend. See your https://bugs.openjdk.org/browse/JDK-8342959
- Making sure we have tests that would detect vectors that are too long. This would require some padding between the vectors, so that we have some untouched space - and if it does get touched we know that a vector was too long. Does that make sense? This is I guess also a general concern - and would have to be applied to all vector instructions.

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

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