On Fri, 24 Apr 2026 01:02:15 GMT, Dean Long <[email protected]> wrote:
> We also do something similar in Compile::build_start_state(), so does it also > need to be changed? I don't see how things can go wrong there. I added an assert anyway. > If the map changes, which map should we call disconnect_inputs on at line > 721? Only the new map, the old map, or both? Good question. I'm not sure line 721 is required. Late inlining, in general, leaves plenty of dead partially connected nodes behind that are collected when `Compile::inline_incrementally_cleanup()` is called. ------------- PR Review Comment: https://git.openjdk.org/valhalla/pull/2356#discussion_r3145815306
