RFR: 8356924: RISC-V: Clean up cost for vector instructions

Gui Cao gcao at openjdk.org
Wed May 14 07:50:52 UTC 2025


On Wed, 14 May 2025 04:04:11 GMT, Dingli Zhang <dzhang at openjdk.org> wrote:

> 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.1.0 with UseRVV (ubuntu24.10):
> * [x]  Run test/jdk/jdk/incubator/vector (fastdebug)
> * [x]  Run test/hotspot/jtreg/compiler/vectorapi (fastdebug)

LGTM.

-------------

Marked as reviewed by gcao (Author).

PR Review: https://git.openjdk.org/jdk/pull/25221#pullrequestreview-2839130010


More information about the hotspot-compiler-dev mailing list