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