RFR: 8309332: RISC-V: Improve PrintOptoAssembly output of vector nodes

Gui Cao gcao at openjdk.org
Fri Jun 2 02:15:26 UTC 2023


Hi, Currently in the vector node implementation, some of the instruction formats are part of the rvv assembly instructions, for which the assembly instructions are also incomplete, such as the missing vsetvli assembly instruction(vsetvli is used to set the element width, number, etc), This makes the compiler assembly output by -XX:+PrintOptoAssembly difficult to understand. And if every assembly instruction is reflected, it will be redundant and increase the maintenance work in the future. And referring to other CPUs, such as the ARM64, it is straightforward to use the instruct function name to simplify [1].

While this won't affect release build, we should fix this for debug build. We can use the -XX:+PrintOptoAssembly parameter to print the compiler assembly output by -XX:+PrintOptoAssembly and view the assembly logic in conjunction with the source code of the specific vector node.

[1] https://github.com/openjdk/jdk/blob/4460429d7a50b9a7a99058ef4e5ae36fb30b956f/src/hotspot/cpu/aarch64/aarch64_vector.ad#L2842-L2846

### Testing:
qemu with UseRVV:

- [ ] Tier1 tests (release)
- [x] test/jdk/jdk/incubator/vector (fastdebug)
- [x] test/jdk/jdk/incubator/vector/Int256VectorTests.java (fastdebug with -XX:+PrintOptoAssembly)

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

Commit messages:
 - RISC-V: Improve PrintOptoAssembly output of vector nodes

Changes: https://git.openjdk.org/jdk/pull/14279/files
 Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=14279&range=00
  Issue: https://bugs.openjdk.org/browse/JDK-8309332
  Stats: 91 lines in 1 file changed: 0 ins; 21 del; 70 mod
  Patch: https://git.openjdk.org/jdk/pull/14279.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/14279/head:pull/14279

PR: https://git.openjdk.org/jdk/pull/14279


More information about the hotspot-compiler-dev mailing list