RFR: 8292289: [vectorapi] Improve the implementation of VectorTestNode [v2]

Thu Aug 18 05:35:22 UTC 2022

On Thu, 18 Aug 2022 03:18:10 GMT, Xiaohong Gong <xgong at openjdk.org> wrote:

>> Quan Anh Mai has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 17 commits:
>> 
>>  - redo aarch
>>  - Merge branch 'master' into improveVTest
>>  - delete aarch64 vector files
>>  - copyright
>>  - fix condition
>>  - fix alltrue
>>  - change VectorTest to subclass Cmp
>>  - add test
>>  - fix aarch build
>>  - remove dst in x86.ad
>>  - ... and 7 more: https://git.openjdk.org/jdk/compare/0d96546a...cac9948e
>
> src/hotspot/share/opto/vectorIntrinsics.cpp line 1810:
> 
>> 1808:     opd2 = opd1;
>> 1809:   }
>> 1810:   Node* cmp = gvn.transform(new VectorTestNode(opd1, opd2, booltest));
> 
> should be `gvn()` instead of `gvn` ?

Fixed, the remaining failure in GHA seems to be JDK-8292498

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

PR: https://git.openjdk.org/jdk/pull/9855