Integrated: 8326438: C2: assert(ld->in(1)->Opcode() == Op_LoadN) failed: Assumption invalid: input to DecodeN is not LoadN

[Daniel Lundén]

On Thu, 21 Mar 2024 15:31:58 GMT, Daniel Lundén <dlunden at openjdk.org> wrote:

> The [`assert`](https://github.com/openjdk/jdk/blob/6f2676dc5f09d350c359f906b07f6f6d0d17f030/src/hotspot/share/opto/graphKit.cpp#L1567) added in [JDK-8310524](https://bugs.openjdk.org/browse/JDK-8310524) is too strong and may sometimes not hold due to the [GVN transformation in `LoadNode::make`](https://github.com/openjdk/jdk/blob/8cb9b479c529c058aee50f83920db650b0c18045/src/hotspot/share/opto/memnode.cpp#L973).
> 
> ### Changeset
> Remove the `assert`.
> 
> ### Testing
> N/A

This pull request has now been integrated.

Changeset: 0c1b254b
Author:    Daniel Lundén <dlunden at openjdk.org>
Committer: Tobias Hartmann <thartmann at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/0c1b254be9ddd3883313f80b61229eacf09aa862
Stats:     4 lines in 1 file changed: 2 ins; 0 del; 2 mod

8326438: C2: assert(ld->in(1)->Opcode() == Op_LoadN) failed: Assumption invalid: input to DecodeN is not LoadN

Reviewed-by: kvn, rcastanedalo, thartmann

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

PR: https://git.openjdk.org/jdk/pull/18434