RFR: 8368095: [leyden] Fix klass dependency recording [v2]
Aleksey Shipilev
shade at openjdk.org
Fri Sep 19 13:49:16 UTC 2025
> See the bug for symptoms. https://github.com/openjdk/leyden/commit/7b7648a4c9f67be509c6fccbcbc0502648388fdc exposed that our dependency recording is not actually accurate. Now that we check that precompiled tasks have dependencies recorded, before we treat class IK as fully initialized, _missing dependencies_ lead to premature replacement of AP4 -> A4, and potential trap from A4.
>
> I see there are two missing things:
> 1. We want to record dependencies on object klasses as well, not only on plain metadata.
> 2. We want to record dependencies even when `cik->is_initialized()` returns `false` at the moment.
>
> Additional testing:
> - [x] Linux x86_64 server fastdebug, `runtime/cds`
> - [ ] Benchmarks
Aleksey Shipilev has updated the pull request incrementally with one additional commit since the last revision:
Retract some unneccesary paths
-------------
Changes:
- all: https://git.openjdk.org/leyden/pull/97/files
- new: https://git.openjdk.org/leyden/pull/97/files/aba729b5..cb48aaca
Webrevs:
- full: https://webrevs.openjdk.org/?repo=leyden&pr=97&range=01
- incr: https://webrevs.openjdk.org/?repo=leyden&pr=97&range=00-01
Stats: 10 lines in 2 files changed: 0 ins; 4 del; 6 mod
Patch: https://git.openjdk.org/leyden/pull/97.diff
Fetch: git fetch https://git.openjdk.org/leyden.git pull/97/head:pull/97
PR: https://git.openjdk.org/leyden/pull/97
More information about the leyden-dev
mailing list