Integrated: 8343827: RISC-V: set AlignVector as false if applicable to enable SLP
Hamlin Li
mli at openjdk.org
Wed Nov 20 08:52:27 UTC 2024
On Fri, 8 Nov 2024 11:40:46 GMT, Hamlin Li <mli at openjdk.org> wrote:
> Hi,
> Can you help to review this patch?
>
> SLP vectorization in C2 checks the data alignment optionally, if fast misaliged access is supported by CPU it will skip the alignment check in SLP, this is good for both C2 compile time and run time (as more loops can be potentially vectorized. And the performance of vector implementation is good enough, which was already verified by existing tests (array copy, crc32, ...)).
> On the other hand, we have some existing tests which verify that generated IRs should contain some specific Nodes, with the intrdocution of Compact Object Header, some tests related to SLP was observed failure which is due to the data offset change which is due to header change in the JEP, by enabling C2 SLP when fast misaliged access is supported, we should be able to resolve these issues. Although riscv support misaligned access optionally, started from RVA22 it's mandatory.
>
> Thanks!
This pull request has now been integrated.
Changeset: 587f2b4b
Author: Hamlin Li <mli at openjdk.org>
URL: https://git.openjdk.org/jdk/commit/587f2b4b4dd73733a6ee247200371f8a8d0299c1
Stats: 6 lines in 1 file changed: 1 ins; 0 del; 5 mod
8343827: RISC-V: set AlignVector as false if applicable to enable SLP
Reviewed-by: fyang, luhenry
-------------
PR: https://git.openjdk.org/jdk/pull/21974
More information about the hotspot-dev
mailing list