RFR: 8340602: C2: LoadNode::split_through_phi might exhaust nodes in case of base_is_phi [v4]

[Daohan Qu]

On Mon, 30 Sep 2024 07:29:22 GMT, Christian Hagedorn <chagedorn at openjdk.org> wrote:

>> Daohan Qu has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Add jtreg requirements and fix some format issues
>
> No worries! Take your time, there is no rush :-) Thanks for letting me know.

Hi, @chhagedorn, I just update with a minimal fix, which:
1. Doesn't affect the execution that didn't cause infinite split before this patch
2. Avoids infinite splitting for the execution that triggered it before and allow them to be compiled by C2

This fix makes some of [code above](https://github.com/quadhier/jdk/blob/2b2c7d5b3e8193db706470725743003a5d25759c/src/hotspot/share/opto/memnode.cpp#L1683-L1710) dead. If this fix makes sense, I could do some cleanup.

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

PR Comment: https://git.openjdk.org/jdk/pull/21134#issuecomment-2416158202