RFR: 8298088: RISC-V: Make Address a discriminated union internally [v2]

Aleksey Shipilev shade at openjdk.org
Thu Dec 15 19:13:06 UTC 2022


On Mon, 5 Dec 2022 13:57:12 GMT, Fei Yang <fyang at openjdk.org> wrote:

>> The RISC-V port has the same issue as: https://bugs.openjdk.org/browse/JDK-8297830
>> It looks to me that the fix for the AArch64 port is a nice refactoring work.
>> This fixes this issue for the RISC-V port with a similar approach.
>> 
>> Testing:
>>   Tier1 tested with release build on linux-riscv64 unmatched board.
>>   Run non-trivial benchmark workloads with fastdebug builds.
>
> Fei Yang has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Review

AArch64 patch (https://github.com/openjdk/jdk/commit/5a5ced3a900a81fd0b0757017f4138ce97e2521e) also has modifications that rewrite `_offset` -> `offset()`, `_base` -> `base()`, `_index` -> `index()`, etc. Does RISC-V code has the relevant uses too?

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

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


More information about the hotspot-compiler-dev mailing list