RFR: JDK-8316991: Reduce nullable allocation merges [v9]

Cesar Soares Lucas cslucas at openjdk.org
Thu Mar 28 19:56:34 UTC 2024


On Wed, 27 Mar 2024 17:33:29 GMT, Vladimir Ivanov <vlivanov at openjdk.org> wrote:

>> Cesar Soares Lucas has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 13 commits:
>> 
>>  - Merge remote-tracking branch 'origin/master' into ram-nullables
>>  - Catching up with master
>>  - Fix broken build.
>>  - Merge with origin/master
>>  - Update test/micro/org/openjdk/bench/vm/compiler/AllocationMerges.java
>>    
>>    Co-authored-by: Andrey Turbanov <turbanoff at gmail.com>
>>  - Ammend previous fix & add repro tests.
>>  - Fix to prevent reducing already reduced Phi
>>  - Fix to prevent creating NULL ConNKlass constants.
>>  - Refrain from RAM of arrays and Phis controlled by Loop nodes.
>>  - Fix typo in test.
>>  - ... and 3 more: https://git.openjdk.org/jdk/compare/89e0889a...3129378f
>
> src/hotspot/share/opto/memnode.cpp line 1552:
> 
>> 1550: // by 'split_through_phi'. The first use of this method was in EA code as part
>> 1551: // of simplification of allocation merges.
>> 1552: // Some differences from original method:
> 
> What "original method" do you refer to here?

The current method, `can_split_through_phi_base`, is just a helper that contains a copy of the validations performed by `split_through_phi`. The original method is `split_through_phi`.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/15825#discussion_r1543564696


More information about the hotspot-dev mailing list