[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