RFR: 8353225: Add a way to iterate Klass inside the loaded CDS archive

[Ioi Lam]

On Sat, 29 Mar 2025 07:53:37 GMT, Thomas Stuefe <stuefe at openjdk.org> wrote:

> Please consider.
> 
> This patch adds a way to iterate all Klass structures in the loaded CDS archive (`SystemDictionaryShared::iterate_klasses_in_shared_archive(ConstKlassClosure* klassClosure, bool is_static);`). This uses the same mechanism that `PrintSharedArchiveAndExit` uses. I  need this mechanism for some future GC work.
> 
> Then, it re-implements `PrintSharedArchiveAndExit` to reuse the new iteration API instead. This also serves as a way to test, since there are pre-existing tests for this flag.
> 
> It also adds a new, rather basic, gtest.
> 
> Finally, the new gtest picked up a small pre-existing bug in the implementation of `PrintSharedArchiveAndExit`: it prints all array classes of all InstanceKlass inside the archive, for all dimensions, without testing if that array class is part of the archive. The array class may have been created dynamically, however. The patch fixes that.
> 
> ----
> 
> Testing:
> - I ensured that the output of `PrintSharedArchiveAndExit`, before and after this patch, are identical apart from the small error fix concerning array classes
> - I ran all runtime/cds tests on Linux x64
> - GHAs

src/hotspot/share/cds/lambdaProxyClassDictionary.cpp line 93:

> 91:     while (k != nullptr) {
> 92:       cl->do_klass(k);
> 93:       k = k->next_link();

When do you plan to use this API? `k->next_link()` will change if `k` has been loaded in runtime, so you can no longer use it to find other archived lambda proxy classes.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/24311#discussion_r2021290384