RFR: 8332827: [REDO] C2: crash in compiled code because of dependency on removed range check CastIIs [v2]

[Emanuel Peter]

On Thu, 12 Dec 2024 08:58:59 GMT, Roland Westrelin <roland at openjdk.org> wrote:

>> Thanks for the updates, @rwestrel !
>> The fix looks reasonable. You can add an extra comment for the precedence edges for Div.
>> I launched some testing now, feel free to ping me later.
>
>> Thanks for the updates, @rwestrel ! The fix looks reasonable. You can add an extra comment for the precedence edges for Div. I launched some testing now, feel free to ping me later.
> 
> Thanks for reviewing this. Would it be possible to run performance testing again? Performance testing was run for the initial fix (that was backed out) and this one is slightly different.

@rwestrel the tests I launched yesterday look clean.

I can run some performance testing. It will take a while to build, launch and run.

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

PR Comment: https://git.openjdk.org/jdk/pull/22568#issuecomment-2540671420