On Thu, 14 Oct 2021 17:21:25 GMT, Paul Sandoz <psan...@openjdk.org> wrote:
>> This PR improves the performance of vector operations that accept masks on >> architectures that support masking in hardware, specifically Intel AVX512 >> and ARM SVE. >> >> On architectures that do not support masking in hardware the same technique >> as before is applied to most operations, specifically composition using >> blend. >> >> Masked loads/stores are a special form of masked operation that require >> additional care to ensure out-of-bounds access throw exceptions. The range >> checking has not been fully optimized and will require further work. >> >> No API enhancements were required and only a few additional tests were >> needed. > > Paul Sandoz has updated the pull request incrementally with one additional > commit since the last revision: > > Apply patch from https://github.com/openjdk/panama-vector/pull/152 > @njian there is a conflict with `macroAssembler_aarch64.cpp`: > > I think the resolution is this: > > ``` > @@ -2581,7 +2572,7 @@ void > MacroAssembler::pop_call_clobbered_registers_except(RegSet exclude) { > > void MacroAssembler::push_CPU_state(bool save_vectors, bool use_sve, > int sve_vector_size_in_bytes, int > total_predicate_in_bytes) { > - push(0x3fffffff, sp); // integer registers except lr & sp > + push(RegSet::range(r0, r29), sp); // integer registers except lr & sp > ``` > > is that correct? Yes, I think so. ------------- PR: https://git.openjdk.java.net/jdk/pull/5873