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

[Xiaohong Gong]

On Thu, 20 Mar 2025 01:47:13 GMT, Xiaohong Gong <xgong 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!
>
> Hi @eme64 , may I know what the test's status is? Seems the failure on linux-x64 is not caused by this PR. Is it possible to move on? Please let me know if any other issues. Thanks a lot!

> @XiaohongGong Can you please merge with master before I launch testing?

Sure, I will do a merge soon. Thanks!

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

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