RFR: 8349361: C2: RShiftL should support all applicable transformations that RShiftI does [v7]
Roland Westrelin
roland at openjdk.org
Fri Feb 14 15:57:54 UTC 2025
On Thu, 13 Feb 2025 10:46:25 GMT, Emanuel Peter <epeter at openjdk.org> wrote:
> I left some comments / suggestions.
Thanks for reviewing this. I pushed new commits that I think cover your comments.
> I'm also wondering about testing. How good do you think test coverage is? Are all cases covered? How about the edge-cases? Could we improve the coverage with randomization somehow?
Transformations that I refactored and now apply to both the long and int `RShift` should now have test cases. I tweaked the test cases so there are now randomized.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/23438#issuecomment-2659694153
More information about the hotspot-compiler-dev
mailing list