Withdrawn: 8273332: [Vector API] C2 fails to check whether the rotate operation is directly supported by the target ISA after JDK-8271366
Jie Fu
jiefu at openjdk.java.net
Sun Sep 5 14:11:52 UTC 2021
On Fri, 3 Sep 2021 15:36:18 GMT, Jie Fu <jiefu at openjdk.org> wrote:
> Hi all,
>
> A lot of Vector API tests from Panama's vectorIntrinsics+mask branch crash [1].
>
> This is because `arch_supports_vector()` fails to check whether the rotate operation is directly supported by the target ISA after JDK-8271366.
> It can be fixed by adding `Matcher::match_rule_supported_vector` check in `inline_vector_broadcast_int()` and `inline_vector_nary_operation()` to prevent generation of unsupported rotate operation IR patterns.
>
> Maybe it's hard to reproduce with the jdk-repo, but the bug still be there in theory.
> So it would be better to fix it in the jdk repo.
>
> Testing:
> - jdk/incubator/vector/ with `-ea -esa -Xcomp -XX:CompileThreshold=100`, the crash reported in [1] gets fixed with Panama's vectorIntrinsics+mask branch
>
> Thanks.
> Best regards,
> Jie
>
> [1] https://bugs.openjdk.java.net/browse/JDK-8273205
This pull request has been closed without being integrated.
-------------
PR: https://git.openjdk.java.net/jdk/pull/5364
More information about the hotspot-compiler-dev
mailing list