RFR: 8302459: Missing late inline cleanup causes compiler/vectorapi/VectorLogicalOpIdentityTest.java IR failure

[Damon Fenacci]

On Thu, 7 Nov 2024 00:01:01 GMT, Vladimir Ivanov <vlivanov at openjdk.org> wrote:

> A proper fix would be to re-examine failed intrinsics call site during IGVN and repeat intrinsifcation attempt when their inputs improve

In the end I've reimplemented the fix with this approach: keeping track of failed late inlining and re-scheduling them in IGVN.  
One potential issue is that there might be useless redundant inlining attempts. This doesn't seem to be much of a problem though: late inlining attempts should be rather rare (only applied to a few invokedynamic, method handles, vector API). Benchmark measurements (DaCapo, Renaissance) for running time and compilation don't show any noticeable difference.

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

PR Comment: https://git.openjdk.org/jdk/pull/21682#issuecomment-2516389661