[lworld] Integrated: 8351006: [lworld] Refactor LoadNode::Identity of an InlineType into can_see_stored_value

Quan Anh Mai qamai at openjdk.org
Tue Mar 11 17:33:07 UTC 2025


On Sun, 2 Mar 2025 07:19:58 GMT, Quan Anh Mai <qamai at openjdk.org> wrote:

> Hi,
> 
> This patch moves the logic of `LoadNode::Identity` that tries to look through an `InlineTypeNode` to `can_see_stored_value`. This also fixes 2 issues:
> 
> - `Identity` should not return a new node
> - `LoadUS` from a `short` is not a mismatched access and hence can return a wrong value
> 
> Please take a look and leave your reviews, thanks a lot.

This pull request has now been integrated.

Changeset: b8b1fbcf
Author:    Quan Anh Mai <qamai at openjdk.org>
Committer: Tobias Hartmann <thartmann at openjdk.org>
URL:       https://git.openjdk.org/valhalla/commit/b8b1fbcfd6e6829c0b7d0ff79869941e303a99fa
Stats:     52 lines in 2 files changed: 35 ins; 17 del; 0 mod

8351006: [lworld] Refactor LoadNode::Identity of an InlineType into can_see_stored_value

Reviewed-by: chagedorn

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

PR: https://git.openjdk.org/valhalla/pull/1387


More information about the valhalla-dev mailing list