[RyuJIT] Don't optimize "x < 0" to shift for unsigned cmp

[EgorBo]

Fixes #42719 (see #35627 (comment))

It used to optimize (-1 u< 0) to (-1 >> 31)

/cc @sandreenko

[sandreenko]

Could you please mark it as fixing #35627 and remove the exclusion for the test at https://github.com/sandreenko/runtime/blob/8c7ecdb1fc5d4ccb6ac667d53255373ae32362b8/src/libraries/Microsoft.CSharp/tests/IntegerBinaryOperationTests.cs#L501?

[sandreenko]

PTAL @CarolEidt cc @dotnet/jit-contrib

[sandreenko]

LGTM with this and a stress library run

Could you please mark it as fixing #35627 and remove the exclusion for the test at https://github.com/sandreenko/runtime/blob/8c7ecdb1fc5d4ccb6ac667d53255373ae32362b8/src/libraries/Microsoft.CSharp/tests/IntegerBinaryOperationTests.cs#L501?

[ViktorHofer]

// Auto-generated message

69e114c which was merged 12/7 removed the intermediate src/coreclr/src/ folder. This PR needs to be updated as it touches files in that directory which causes conflicts.

To update your commits you can use this bash script: https://gist.github.com/ViktorHofer/6d24f62abdcddb518b4966ead5ef3783. Feel free to use the comment section of the gist to improve the script for others.

[EgorBo]

@dotnet/jit-contrib can this be merged?