RFR: 8323429: Missing C2 optimization for FP min/max when both inputs are same [v3]

Christian Hagedorn chagedorn at openjdk.org
Wed Apr 17 12:23:02 UTC 2024


On Wed, 17 Apr 2024 11:08:01 GMT, Galder Zamarreño <galder at openjdk.org> wrote:

>> Added C2 identity optimization for min/max calls, whereby if both inputs are the same, either is returned.
>> 
>> It includes an IR test to verify that the optimization gets applied. The optimization applies not only to floating points, but also long and ints. The test includes tests for all of those.
>> 
>> `BasicDoubleOpTest.vectorMax_8322090` has also been adjusted to match expectations after implementing the optimization.
>> 
>> I've run hotspot compiler tests successfully on x86_64.
>
> Galder Zamarreño has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Use failOn instead of counts

Testing passed, looks good!

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

Marked as reviewed by chagedorn (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/18738#pullrequestreview-2005890959


More information about the hotspot-compiler-dev mailing list