On Tue, 28 Apr 2026 09:59:41 GMT, Roland Westrelin <[email protected]> wrote:
>> When doing late inlining, `GraphKit::access_load_at()` may be called >> to populate an `InlineTypeNode`. When it executes, it, sometimes, >> causes the `map` used by the `GraphKit` instance to change. When that >> happens, `CallGenerator::do_late_inline_helper()` continues execution >> with an outdated `map` which causes issues during parsing. >> >> --------- >> - [x] I confirm that I make this contribution in accordance with the >> [OpenJDK Interim AI Policy](https://openjdk.org/legal/ai). > > Roland Westrelin has updated the pull request incrementally with one > additional commit since the last revision: > > review Marked as reviewed by dlong (no project role). OK, I think GraphKit::SavedState could be rewritten to keep the same map, but I think it would be a much bigger and riskier change. ------------- PR Review: https://git.openjdk.org/valhalla/pull/2356#pullrequestreview-4194861215 PR Comment: https://git.openjdk.org/valhalla/pull/2356#issuecomment-4341652171
