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