[riscv-port] RFR: 8278994: riscv: RVC support [v8]

Fei Yang fyang at openjdk.java.net
Thu Jan 6 09:54:46 UTC 2022


On Thu, 6 Jan 2022 09:30:19 GMT, Xiaolin Zheng <xlinzheng at openjdk.org> wrote:

>> Hi team,
>> 
>> This patch includes the basic definition of the RVC instruction set and some cleanups. Tested a simple `test/hotspot/jtreg/compiler/` folder on qemu.
>> 
>> Using `<JAVA_HOME>/bin/java -XX:+UnlockExperimentalVMOptions -XX:+UseRVC -XX:+UnlockDiagnosticVMOptions -XX:+PrintAssembly -XX:PrintAssemblyOptions=no-aliases,numeric -XX:+PrintStubCode -XX:-TieredCompilation` could show RVC instructions.
>> 
>> Thanks,
>> Xiaolin
>
> Xiaolin Zheng has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Remove remaining macros as discussions

src/hotspot/cpu/riscv/assembler_riscv.hpp line 1345:

> 1343:   protected:
> 1344:     Assembler *_masm;
> 1345:     bool _prev_in_compressible_region;

suggestion: rename to "_saved_in_compressible_region"

src/hotspot/cpu/riscv/assembler_riscv.hpp line 1858:

> 1856: 
> 1857: // --------------------------
> 1858: // sub/subw -> c.sub/c.subw

It's better to move this comment before the if block which does compression.

src/hotspot/cpu/riscv/assembler_riscv.hpp line 1898:

> 1896: private:
> 1897: // some helper functions
> 1898:   bool check_rvc() const {

maybe rename to "bool do_compress"?

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

PR: https://git.openjdk.java.net/riscv-port/pull/34


More information about the riscv-port-dev mailing list