XiaohongGong
XiaohongGong
> The performance improvement is impressive on x86 with avx2. > > Can you clarify what tests did you do? Thanks. Hi, thanks for looking at this PR! To test...
Hi, the new commit reverted the changes to unify the vector mask cast operation with `VectorMaskCast` op, so that the whole patch can focus on fixing the performance issue on...
> Thanks for the update. > > May I ask can we do it like this? > > ``` > diff --git a/src/hotspot/share/opto/vectorIntrinsics.cpp b/src/hotspot/share/opto/vectorIntrinsics.cpp > index 66bacf0..c4e807a 100644 > ---...
> LGTM Thanks for the update. Thanks so much for the review!
Hi @jatin-bhateja, could you please help to take a look at this PR? Thanks so much for your time!
ping again. Could anyone else please help to take a look at this PR? Thanks a lot!
The GHA failure is not related to this PR. So may I get an approve from someone else? Thanks in advance!
Please rebase this PR and resolve the conflicts. Then I can have an internal tests for the aarch64 change parts. Thanks a lot!
Tests pass on aarch64 NEON and SVE. So aarch64 part looks good to me. Thanks!
/label add hotspot-compiler