RFR: 8264320: ShouldNotReachHere in Compile::print_inlining_move_to()

Vladimir Ivanov vlivanov at openjdk.java.net
Fri Apr 23 12:57:26 UTC 2021

On Thu, 22 Apr 2021 16:56:57 GMT, Vladimir Ivanov <vlivanov at openjdk.org> wrote:

> Cloned `CallNode` gets a fresh `CallGenerator` which has to be put on `_print_inlining_list` when -XX:+PrintInlining is used.
> Otherwise, if the  `CallGenerator` ends up on `_late_inlines` list (which is processed by `Compile::process_late_inline_calls_no_inline()`, `Compile::print_inlining_move_to()` can't locate it in `_print_inlining_list` and triggers the crash. 
> The fix is to put cloned `CallGenerator` next to original one in `_print_inlining_list`.
> Testing:
> - [x] failing test
> - [x] hs-tier1 - hs-tier4

Thanks for the reviews, Vladimir and Tobias.


PR: https://git.openjdk.java.net/jdk/pull/3635

More information about the hotspot-compiler-dev mailing list