RFR: 8341293: Split field loads through Nested Phis [v2]

[Dhamoder Nalla]

On Wed, 22 Jan 2025 07:09:50 GMT, Emanuel Peter <epeter at openjdk.org> wrote:

>> Thanks @eme64 & @chhagedorn ,  added IR rule for 64bit and EliminateAllocations.
>> 
>> Can you please help with equivalent IR rules for the vm flags  vm.debug == true & vm.compiler2.enabled 
>> 
>> I tried the following, but they did not work
>> applyIfAnd = {"debug", "true", "compiler2.enabled", "true"}
>
> @dhanalla The IR framework only checks the `@IR` rules if we are in debug more (`vm.debug == true`) and if we use C2 (`vm.compiler2.enabled`). So I think you can just drop these and see if that works!

Thanks @eme64, I couldn't drop (vm.debug == true) because the test scenarios use some of the VM options (DeoptimizeALot, TraceReduceAllocationMerges) that are only available in the debug build. The test fails in the release build if I drop it.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/21270#discussion_r1927857535