[lworld] Integrated: 8374786: [lworld] C2: Missing asserts in parse3.cpp after merging JDK-8341411
Christian Hagedorn
chagedorn at openjdk.org
Wed Jan 28 07:09:31 UTC 2026
On Tue, 27 Jan 2026 13:45:00 GMT, Christian Hagedorn <chagedorn at openjdk.org> wrote:
> When merging in [JDK-8341411](https://bugs.openjdk.org/browse/JDK-8341411), we wrongly dropped the following two similar asserts:
>
> assert(C->get_alias_index(adr_type) == C->get_alias_index(_gvn.type(adr)->isa_ptr()),
> "slice of address and input slice don't match");
>
> This was difficult to spot because `adr` was only defined further down in the code at the time of the merge. [JDK-8357474](https://bugs.openjdk.org/browse/JDK-8357474) moved `adr` up to the place where we also define it in mainline which is how it was noticed that they are missing. This patch simply adds the asserts again
>
> Thanks,
> Christian
This pull request has now been integrated.
Changeset: 90cc515c
Author: Christian Hagedorn <chagedorn at openjdk.org>
URL: https://git.openjdk.org/valhalla/commit/90cc515cff2a4ed34a9a25bdea8e88f391e64bd4
Stats: 4 lines in 1 file changed: 4 ins; 0 del; 0 mod
8374786: [lworld] C2: Missing asserts in parse3.cpp after merging JDK-8341411
Reviewed-by: liach, thartmann
-------------
PR: https://git.openjdk.org/valhalla/pull/1982
More information about the valhalla-dev
mailing list