RFR: 8332268: C2: Add missing optimizations for UDivI/L and UModI/L and unify the shared logic with the signed nodes [v3]

[Quan Anh Mai]

On Mon, 25 Nov 2024 14:20:15 GMT, theoweidmannoracle <duke at openjdk.org> wrote:

>> That may be true. But `uncast` would be still a good idea, so that we can see through `Cast` nodes. I think it could then also be added to `ModI/L`.
>> 
>> @merykitty do you have an easy way to unit test this, i.e. to have a test with a Cast?
>
> @eme64 I didn't mean to imply it's not useful. But I think it would be better to do these changes to the existing code for ModI/L and DivI/L in a separate RFE.

I don't see the logic in "Doing it the same as `ModI/LNode` then changing all of them together" instead of "Doing the new thing in the better way then changing the old thing to match it"

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

PR Review Comment: https://git.openjdk.org/jdk/pull/22061#discussion_r1856712025