8267370: [Vector API] Fix several crashes after JDK-8256973 #4109
@DamonFool This change now passes all automated pre-integration checks.
ℹ️ This project also has non-automated pre-integration requirements. Please see the file CONTRIBUTING.md for details.
After integration, the commit message for the final commit will be:
At the time when this comment was updated there had been 41 new commits pushed to the
As there are no conflicts, your changes will automatically be rebased on top of these commits when integrating. If you prefer to avoid this automatic rebasing, please check the documentation for the /integrate command for further details.
➡️ To integrate this PR with the above commit message to the
Hi @DamonFool ,
Thanks much for providing a quick fix. Problem here seems to be related to assert in following assembler routines which made an assumption that the instruction is only supported over AVX2 platforms.
But, its ok to limit the patch to AVX2 considering optimization is majorly aimed at modern server targets (which should supported AVX2).
An alternate fix is proposed below which does not restrict the optimization for AVX2 and still keep the changes minimal, since movmaskb is already used at several places in context of 256 bit vector argument.
Good catch! @jatin-bhateja
More testing is in progress.
@DamonFool Since your change was applied there have been 41 commits pushed to the
Your commit was automatically rebased without conflicts.
Pushed as commit 7a63ff7.
💡 You may see a message that your pull request was closed with unmerged commits. This can be safely ignored.