Integrated: 8310524: C2: record parser-generated LoadN nodes for IGVN

Daniel Lundén duke at openjdk.org
Fri Dec 8 11:07:30 UTC 2023


On Tue, 5 Dec 2023 09:05:35 GMT, Daniel Lundén <duke at openjdk.org> wrote:

> This changeset fixes an issue where LoadN nodes were not recorded during bytecode parsing for later revisit in IGVN, in some cases resulting in missed optimization opportunities (see, e.g., the included new regression test).
> 
> Changes:
> - Make sure to record newly added LoadN-nodes for IGVN in `GraphKit::make_load`.
> - Add a regression test.
> 
> ### Testing
> - tier1, tier2, tier3, tier4, tier5 (windows-x64, linux-x64, linux-aarch64, macosx-x64, macosx-aarch64)

This pull request has now been integrated.

Changeset: 9e48b90c
Author:    Daniel Lundén <daniel.lunden at oracle.com>
Committer: Roberto Castañeda Lozano <rcastanedalo at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/9e48b90c7fd349195a1389c480c66dfd9b1a7f75
Stats:     68 lines in 2 files changed: 68 ins; 0 del; 0 mod

8310524: C2: record parser-generated LoadN nodes for IGVN

Reviewed-by: chagedorn, rcastanedalo, thartmann

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

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


More information about the hotspot-compiler-dev mailing list