RFR: 8349522: AArch64: Add backend implementation for new unsigned and saturating vector operations [v2]

[Xiaohong Gong]

On Thu, 13 Mar 2025 09:32:20 GMT, Emanuel Peter <epeter at openjdk.org> wrote:

> I'm getting this failure with `-XX:UseAVX=1` on x64. It is a new test you added.
> 
> ```
> Failed IR Rules (1) of Methods (1)
> ----------------------------------
> 1) Method "public void compiler.vectorapi.VectorSaturatedOperationsTest.susub_masked()" - [Failed IR rules: 1]:
>    * @IR rule 1: "@compiler.lib.ir_framework.IR(phase={BEFORE_MATCHING}, applyIfPlatformAnd={}, applyIfCPUFeatureOr={"avx", "true", "asimd", "true"}, counts={"_#V#SATURATING_SUB_VL#_", " >0 ", "unsigned_vector_node", " >0 "}, failOn={}, applyIfPlatform={}, applyIfPlatformOr={}, applyIfOr={}, applyIfCPUFeatureAnd={}, applyIf={}, applyIfCPUFeature={}, applyIfAnd={}, applyIfNot={})"
>      > Phase "Before matching":
>        - counts: Graph contains wrong number of nodes:
>          * Constraint 1: "(\\d+(\\s){2}(SaturatingSubV.*)+(\\s){2}===.*vector[A-Za-z]<J,2>)"
>            - Failed comparison: [found] 0 > 0 [given]
>            - No nodes matched!
>          * Constraint 2: "unsigned_vector_node"
>            - Failed comparison: [found] 0 > 0 [given]
>            - No nodes matched!
> ```

I will take a look at it. Thanks for your reporting!

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

PR Comment: https://git.openjdk.org/jdk/pull/23608#issuecomment-2720650861