RFR: 8351833: Unexpected increase in live nodes when splitting Phis through MergeMems in PhiNode::Ideal [v2]

Daniel Lundén dlunden at openjdk.org
Thu Apr 10 11:43:40 UTC 2025


On Mon, 7 Apr 2025 12:36:57 GMT, Christian Hagedorn <chagedorn at openjdk.org> wrote:

>> Daniel Lundén has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Clean up code related to eager transformation and rename itergvn to igvn
>
> test/hotspot/jtreg/TEST.groups line 187:
> 
>> 185:   compiler/interpreter/ \
>> 186:   compiler/jvmci/ \
>> 187:   compiler/itergvn/ \
> 
> I suggest to use the short name `igvn`
> 
> On a separate note, I think we should go through all our test folders in `jtreg/compiler` and check if we should add more folders to tier1. For example, `splitif` or `predicates` tests are currently not executed in tier1 but they probably should.

Thanks, fixed! Do you want to create an RFE for adding the additional folders to tier1 or should I create one?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/24325#discussion_r2037152511


More information about the hotspot-compiler-dev mailing list