Hamlin Li
Hamlin Li
Hi, Can you help to review this simple patch trying to https://github.com/luben/zstd-jni/issues/284 in github? (This is the 2nd try of https://github.com/luben/zstd-jni/pull/285) Thanks!
Hi, ( I'm new to lz4-java, please kindly let me know if I should do this in another way. ) This is not an issue report !! :) Just want...
Hi, Can you help to review this patch? Thanks This is a continuation of work based on [1] by @XiaohongGong, most work was done in that pr. In this new...
Hi, Can you review the simple patch? Thanks FYI: Discussed https://github.com/openjdk/jdk/pull/18294#issuecomment-1997727704, this usage of UseNeon flag should be removed, as neon is working by default, and UseNeon could be false,...
Hi, Can you have a review on this patch to add RoundVF/RoundDF intrinsics? Thanks! ## Tests test/hotspot/jtreg/compiler/vectorization/TestRoundVectRiscv64.java test/hotspot/jtreg/compiler/c2/cr6340864/TestFloatVect.java test/hotspot/jtreg/compiler/c2/cr6340864/TestDoubleVect.java test/hotspot/jtreg/compiler/floatingpoint/TestRound.java test/jdk/java/lang/Math/RoundTests.java --------- ### Progress - [ ] Change must be...
HI, Can you have a look at this patch adding some tests for Math.round instrinsics? Thanks! ### FYI: During the development of RoundVF/RoundF, we faced the issues which were only...
Hi, Can you help to reivew this simple patch to remove some wrong instrunctions on riscv? These instrunctions are wrong in that e.g. take `vror.vx` as example, * by definition...
Hi, Can you help to review this patch? For vector shift instruct, some corresponding tests are not enabled, this is to enable them. And the way how vector shift instruct...
Hi, Can you help to review the patch? This pr is based on previous work and discussion in [pr 16234](https://github.com/openjdk/jdk/pull/16234), [pr 18294](https://github.com/openjdk/jdk/pull/18294). Compared with previous prs, the major change in...
Hi, Can you help to review the patch? The motivation is to implement `MulAddVS2VI`. But to enable `MulAddVS2VI`, `MulAddS2I` is prerequisite, although `MulAddS2I` does not bring extra benefit on riscv...