RFR: 8310524: C2: record parser-generated LoadN nodes for IGVN
Christian Hagedorn
chagedorn at openjdk.org
Tue Dec 5 10:01:34 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)
Looks good!
-------------
Marked as reviewed by chagedorn (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/16967#pullrequestreview-1764604357
More information about the hotspot-compiler-dev
mailing list