RFR: 8321003: RISC-V: C2 MulReductionVI [v5]
Hamlin Li
mli at openjdk.org
Wed Feb 19 14:02:55 UTC 2025
On Wed, 19 Feb 2025 00:48:50 GMT, Fei Yang <fyang at openjdk.org> wrote:
>> Yes, you're right, fixed. Thanks!
>
> Thanks for the update. So now the FP reduce multiply will only apply to the Vector-API use case. Did you check the auto-vectorization use case? I suppose the two tests `ProdRed_Double.java` & `ProdRed_Float.java` which are enabled for riscv64 by this PR won't work with this update.
@RealFYang I think in this PR, I need to remove the Reduction intrinsics for V & D, as seems to me there is no way to supply an intrinsic for unordered one, but not for the ordered one, it will lead to AD file error. Or maybe can you suggest how to do it?
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/23580#discussion_r1961733548
More information about the hotspot-compiler-dev
mailing list