RFR: 8292575: riscv: Represent Registers as values [v2]

Fei Yang fyang at openjdk.org
Tue Aug 30 01:00:56 UTC 2022


On Fri, 26 Aug 2022 02:49:17 GMT, Feilong Jiang <fjiang at openjdk.org> wrote:

>> Follow up [JDK-8292153](https://bugs.openjdk.org/browse/JDK-8292153) and [JDK-8292203](https://bugs.openjdk.org/browse/JDK-8292203), migrate away from pointer-based representation of Register values.
>> 
>> It improves compile-time checking by forbidding implicit conversions between integrals and pointers.
>> 
>> Testing:
>> - hotspot/jdk tier1 on unmatched board
>> - hotspot/jdk tier1 on qemu
>
> Feilong Jiang 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 two additional commits since the last revision:
> 
>  - Merge branch 'master' of https://github.com/openjdk/jdk into riscv-register-rep
>  - Represent Registers as values

src/hotspot/cpu/riscv/vmreg_riscv.inline.hpp line 33:

> 31: }
> 32: 
> 33: inline VMReg FloatRegister::FloatRegister::FloatRegisterImpl::as_VMReg() const {

Shouldn't this be: inline VMReg FloatRegister::FloatRegisterImpl::as_VMReg() const?

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

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


More information about the hotspot-dev mailing list