Integrated: 8301740: RISC-V: Address::uses() should check address mode

Dingli Zhang dzhang at openjdk.org
Mon Feb 6 02:43:57 UTC 2023


On Fri, 3 Feb 2023 06:58:42 GMT, Dingli Zhang <dzhang at openjdk.org> wrote:

> Follow up [JDK-8299733](https://bugs.openjdk.org/browse/JDK-8299733). Although RISC-V does not trigger the same issue at the moment, we should better deal with this as well.
> 
> We have only three addressing mode in RISC-V and I think switch statement is not needed. If we get another literal type, we can update here together with many places that would need to be updated at the same time. I think it's enough to just test for non-literal so far.
> 
> ## Testing:
> 
> - all tier1 on unmatched board without new failures

This pull request has now been integrated.

Changeset: 4dd6e8f9
Author:    Dingli Zhang <dzhang at openjdk.org>
Committer: Fei Yang <fyang at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/4dd6e8f9dc98c880b88d86e053da49e260065140
Stats:     1 line in 1 file changed: 0 ins; 0 del; 1 mod

8301740: RISC-V: Address::uses() should check address mode

Reviewed-by: luhenry, fjiang, fyang

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

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


More information about the hotspot-compiler-dev mailing list