[vectorIntrinsics] Integrated: 8261394: [vector] Crash with "assert(Matcher::vector_size_supported(elem_bt, length)) failed: length in range"

[Xiaohong Gong]

On Wed, 10 Feb 2021 06:06:45 GMT, Xiaohong Gong <xgong at openjdk.org> wrote:

> The crash is introduced by [1] and happens when the compiler is making an unspported vector type:
>   assert(Matcher::vector_size_supported(elem_bt, length)) failed: length in range
> Before vectorization for each vector intrinsic, the hotspot will check whether current arch supports the vector operation from several aspects like the backend implementation, the vector type and length. Once one of them is not matched, the compiler will stop the vectorization and go back to the default java implementation.
> 
> The changes in [1] missed the check for "Op_CallLeafVector". I think the double 64-bits vector is not supported to be vectorized. However, due to the missing check, the compiler continues the progress and then the crash happens.
> 
> This patch fixes it by making sure the check contains all opcodes.
> 
> [1] https://bugs.openjdk.java.net/browse/JDK-8261267

This pull request has now been integrated.

Changeset: e901c4d9
Author:    Xiaohong Gong <xgong at openjdk.org>
Committer: Ningsheng Jian <njian at openjdk.org>
URL:       https://git.openjdk.java.net/panama-vector/commit/e901c4d9
Stats:     8 lines in 1 file changed: 7 ins; 0 del; 1 mod

8261394: [vector] Crash with "assert(Matcher::vector_size_supported(elem_bt, length)) failed: length in range"

Reviewed-by: njian, jiefu, sviswanathan

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

PR: https://git.openjdk.java.net/panama-vector/pull/38