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

[Emanuel Peter]

On Wed, 22 Jan 2025 01:28:30 GMT, Dhamoder Nalla <dhanalla at openjdk.org> wrote:

>> Hmm, in my experience these flags are not really necessary, and rather a restriction we would like to remove if possible.
>> 
>> Is there an argument why they cannot be moved to the IR rules with `applyIf...`?
>
> 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!

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

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