On Tue, 27 Jan 2026 13:45:00 GMT, Christian Hagedorn <[email protected]> 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 <[email protected]> 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
