RFR: 8310190: C2 SuperWord: AlignVector is broken, generates misaligned packs [v3]
Emanuel Peter
epeter at openjdk.org
Wed Oct 11 14:34:09 UTC 2023
On Fri, 6 Oct 2023 23:44:02 GMT, Vladimir Kozlov <kvn at openjdk.org> wrote:
>> Emanuel Peter has updated the pull request incrementally with one additional commit since the last revision:
>>
>> fix up TestVectorizationMismatchedAccess.java
>
> src/hotspot/share/opto/compile.cpp line 3668:
>
>> 3666:
>> 3667: case Op_LoadVector:
>> 3668: case Op_StoreVector:
>
> What about Masked Load and Store vectors?
@vnkozlov for now, I only mark the vectors produced from SuperWord. And SuperWord only produces LoadVector/StoreVector, and nothing masked.
The masked vector memops come from the VectorAPI. But the VectorAPI does not respect AlignVector, and may generate misaligned vector ops - and then the verification fails.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/14785#discussion_r1355108445
More information about the hotspot-compiler-dev
mailing list