RFR: 8356924: RISC-V: Clean up cost for vector instructions [v2]
Dingli Zhang
dzhang at openjdk.org
Tue May 20 08:40:35 UTC 2025
> As mentioned in https://bugs.openjdk.org/browse/JDK-8285790 regarding the ARM64 vector instruct modifications:
> Since the new rules are unique and setting different "ins_cost" makes no sense, we have switched to using the default cost.
>
> Currently, there is a similar situation on RISC-V. Over half of the instructions in riscv_v.ad do not include ins_cost definitions. Additionally, as RVV nodes are also unique, we can unify the format by removing these ins_cost entries from riscv_v.ad.
>
> ### Testing
> qemu-system 9.2.3 with UseRVV (ubuntu24.10):
> * [x] Run test/jdk/jdk/incubator/vector (fastdebug)
> * [x] Run test/hotspot/jtreg/compiler/vectorapi (fastdebug)
Dingli Zhang has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains three additional commits since the last revision:
- Remove added ins_cost(VEC_COST) due to merging the main branch
- Merge branch 'master' into master-remove-ins_cost
- 8356924: RISC-V: Clean up cost for vector instructions
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/25221/files
- new: https://git.openjdk.org/jdk/pull/25221/files/80ab16a5..44521d8a
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=25221&range=01
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=25221&range=00-01
Stats: 23889 lines in 747 files changed: 8828 ins; 11491 del; 3570 mod
Patch: https://git.openjdk.org/jdk/pull/25221.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/25221/head:pull/25221
PR: https://git.openjdk.org/jdk/pull/25221
More information about the hotspot-compiler-dev
mailing list