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

[Xiaohong Gong]

On Thu, 18 Aug 2022 02:55:44 GMT, Quan Anh Mai <duke at openjdk.org> wrote:

>> src/hotspot/cpu/aarch64/matcher_aarch64.hpp line 167:
>> 
>>> 165: 
>>> 166:   // Some architecture needs a helper to check for alltrue vector
>>> 167:   static bool vectortest_need_second_argument(bool is_alltrue, bool is_predicate) {
>> 
>> I think we can get the predicate feature in the backend (i.e.  `Matcher::has_predicated_vectors()`), and we do not need the argument here?
>
> On AVX3 the availability of predicated vectors does not ensure that every species of vector mask would be translated into predicated vector. To be more specific, although AVX512F returns true for `Matcher::has_predicated_vectors()`, byte and short vector masks required AVX512BW to be translated into predicated vectors and vector masks of < 512-bit species require AVX512VL.

OK, I see, thanks!

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

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