RFR: 8374789: C2: refactor GraphKit code that create AddP nodes for raw memory to use helper method [v8]

Guanqiang Han ghan at openjdk.org
Mon Feb 23 12:58:11 UTC 2026


On Mon, 23 Feb 2026 07:54:49 GMT, Guanqiang Han <ghan at openjdk.org> wrote:

>> This PR refactors GraphKit code that creates AddP nodes for raw memory to use a dedicated helper method. This avoids passing top() explicitly at call sites and improves readability.
>> 
>> Please review this change. Thanks!
>> 
>> **Test:** GHA
>
> Guanqiang Han has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains 15 additional commits since the last revision:
> 
>  - Merge remote-tracking branch 'upstream/master' into 8374789
>  - Merge two lines into one
>  - Adjust code style again
>  - Adjust code style
>  - Merge remote-tracking branch 'upstream/master' into 8374789
>  - adjust code style
>  - adjust code style
>  - Merge remote-tracking branch 'upstream/master' into 8374789
>  - Align properly
>  - Replace AddPNode constructor calls with static func
>  - ... and 5 more: https://git.openjdk.org/jdk/compare/a21a5814...3d07f2fe

Hi @rwestrel, could you please also take a look at this PR? Thanks!

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

PR Comment: https://git.openjdk.org/jdk/pull/29666#issuecomment-3944633290


More information about the shenandoah-dev mailing list