RFR: 8318723: RISC-V: C2 UDivL
Hamlin Li
mli at openjdk.org
Tue Oct 24 15:29:04 UTC 2023
Hi,
Can you review the change to add intrinsic for UDivI and UDivL?
Thanks!
## Tests
### Functionality
Run tests successfully found via `grep -nr test/jdk/ -we divideUnsigned` and `grep -nr test/hotspot/ -we divideUnsigned`
### Performance
#### Long
** Before **
LongDivMod.testDivideUnsigned 1024 mixed avgt 2 19852.277 ns/op
LongDivMod.testDivideUnsigned 1024 positive avgt 2 29155.681 ns/op
LongDivMod.testDivideUnsigned 1024 negative avgt 2 6385.280 ns/op
** After **
LongDivMod.testDivideUnsigned 1024 mixed avgt 2 11776.806 ns/op
LongDivMod.testDivideUnsigned 1024 positive avgt 2 16101.940 ns/op
LongDivMod.testDivideUnsigned 1024 negative avgt 2 6433.223 ns/op
#### Integer
** Before **
IntegerDivMod.testDivideUnsigned 1024 mixed avgt 2 23498.570 ns/op
IntegerDivMod.testDivideUnsigned 1024 positive avgt 2 16875.614 ns/op
IntegerDivMod.testDivideUnsigned 1024 negative avgt 2 30310.243 ns/op
** After **
IntegerDivMod.testDivideUnsigned 1024 mixed avgt 2 23327.997 ns/op
IntegerDivMod.testDivideUnsigned 1024 positive avgt 2 16708.209 ns/op
IntegerDivMod.testDivideUnsigned 1024 negative avgt 2 30162.153 ns/op
-------------
Commit messages:
- Modify tests to run for riscv64
- reuse riscv_enc_divuw and corrected_idiv
- space and comments
- Initial commit
Changes: https://git.openjdk.org/jdk/pull/16346/files
Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=16346&range=00
Issue: https://bugs.openjdk.org/browse/JDK-8318723
Stats: 74 lines in 5 files changed: 62 ins; 0 del; 12 mod
Patch: https://git.openjdk.org/jdk/pull/16346.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/16346/head:pull/16346
PR: https://git.openjdk.org/jdk/pull/16346
More information about the hotspot-dev
mailing list