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