8265154: vinserti128 operand mix up for KNL platforms #3480
There is a bug in macro assembler in vinserti128 special handling for platforms like KNL that do not support AVX512VL.
Should have been:
The text was updated successfully, but these errors were encountered:
@sviswa7 This change now passes all automated pre-integration checks.
After integration, the commit message for the final commit will be:
At the time when this comment was updated there had been 73 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.
@vnkozlov @TobiHartmann Thanks a lot for the review.
Majority of the places where vinserti128 is used in code gen, the dst and nds are passed as same register. Only in the implementation of VectorInsert node for vector api, the dst and nds are different.
The vector api tests are run as part of tier 3.
@sviswa7 Since your change was applied there have been 74 commits pushed to the
Your commit was automatically rebased without conflicts.
Pushed as commit c108e7a.