RFR: 8318222: RISC-V: C2 CmpU3
Hamlin Li
mli at openjdk.org
Mon Oct 23 15:51:44 UTC 2023
Hi,
Can you review the change to add intrinsic for CmpU3 and CmpUL3?
Thanks!
## Test
### functionality
pass jtreg test:
jdk/java/lang/Long/Unsigned.java, jdk/java/lang/Integer/Unsigned.java
### performance
#### Long
**before**:
Benchmark (size) Mode Cnt Score Error Units
Longs.compareUnsignedDirect 500 avgt 5 1454.789 ± 129.557 ns/op
Longs.compareUnsignedIndirect 500 avgt 5 1410.146 ± 120.017 ns/op
**after**:
Benchmark (size) Mode Cnt Score Error Units
Longs.compareUnsignedDirect 500 avgt 5 1286.129 ± 8.441 ns/op
Longs.compareUnsignedIndirect 500 avgt 5 993.490 ± 0.840 ns/op
#### Integer
**before**:
Benchmark (size) Mode Cnt Score Error Units
Integers.compareUnsignedDirect 500 avgt 5 1611.753 ± 0.700 ns/op
Integers.compareUnsignedIndirect 500 avgt 5 1775.093 ± 1.520 ns/op
**after**:
Benchmark (size) Mode Cnt Score Error Units
Integers.compareUnsignedDirect 500 avgt 5 1159.351 ± 0.601 ns/op
Integers.compareUnsignedIndirect 500 avgt 5 776.185 ± 0.924 ns/op
-------------
Commit messages:
- indent
- rename from 'ui' to 'uw'
- Merge branch 'master' into compareUnsigned
- fix indent
- Initial commit
Changes: https://git.openjdk.org/jdk/pull/16314/files
Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=16314&range=00
Issue: https://bugs.openjdk.org/browse/JDK-8318222
Stats: 67 lines in 3 files changed: 63 ins; 0 del; 4 mod
Patch: https://git.openjdk.org/jdk/pull/16314.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/16314/head:pull/16314
PR: https://git.openjdk.org/jdk/pull/16314
More information about the hotspot-dev
mailing list