[riscv-port] RFR: 8280683: riscv: Remove uses of long and unsigned long

Fei Yang fyang at openjdk.java.net
Fri Jan 28 06:45:38 UTC 2022


On Fri, 28 Jan 2022 05:13:00 GMT, Yadong Wang <yadongwang at openjdk.org> wrote:

> The riscv port should follow https://bugs.openjdk.java.net/browse/JDK-8248404, for the same reason of:
> 
> In many cases the RISCV64 back end uses long and unsigned long types. These types don't work on LLP64 systems (such as Windows) and should be replaced either by int64_t or jlong, as appropriate.
> 
> 
> Hotspot/jdk tier1 passed on the unmatched board. And all jtreg tests have been tested on Qemu without new failures.

Changes requested by fyang (Lead).

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

> 167:   Address(Register r)
> 168:     : _base(r),     _index(noreg), _offset(0), _mode(base_plus_offset), _target(NULL) { }
> 169: 

Suggestion: better to include header file: #include "metaprogramming/enableIf.hpp"

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

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


More information about the riscv-port-dev mailing list