[jdk21u-dev] RFR: 8358534: Bailout in Conv2B::Ideal when type of cmp input is not supported

[Victor Noël]

On Thu, 19 Jun 2025 11:34:42 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:

> Clean bailout from the optimization added in JDK 21. While the patch is fairly recent, it does not come with elevated risk, as it just skips the optimization in the case where the alternative is compiler crash. This crash is seen in real world: https://github.com/adoptium/adoptium-support/issues/1191
> 
> Additional testing:
>  - [x] Linux x86_64 server fastdebug, `all`

Ah ! Thank you both for the explanation, I better understand now (should have thought about double checking the bug itself!). Thanks again for the fix, I will patiently wait then :)

Cheers!

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

PR Comment: https://git.openjdk.org/jdk21u-dev/pull/1904#issuecomment-3079824431