[prev in list] [next in list] [prev in thread] [next in thread]
List: openjdk-hotspot-runtime-dev
Subject: Integrated: 8290739: Simplify storage of dump-time class information
From: Ioi Lam <iklam () openjdk ! org>
Date: 2022-07-27 23:55:07
Message-ID: P16s44EsfY-fu5vb1u2YJMT8QQuMAkVfG6bnSPLmdZ8=.c62bc1ff-b4c6-476a-8f1e-af96855cbe85 () github ! com
[Download RAW message or body]
On Mon, 25 Jul 2022 22:06:51 GMT, Ioi Lam <iklam@openjdk.org> wrote:
> Some code clean up in anticipation of future CDS development.
>
> I moved the initialization of `SystemDictionaryShared::_dumptime_table` and \
> `SystemDictionaryShared::_dumptime_lambda_proxy_class_dictionary` to VM bootstrap, \
> so that:
> - We don't need to allocate these tables dynamically
> - We don't need to check if these tables exist when writing the archive
>
> The current implementation guarantees that \
> `SystemDictionaryShared::_dumptime_table` always contains a `DumpTimeClassInfo` for \
> each `InstanceKlass` that's being considered for inclusion in the CDS archive. I \
> simplified the old `SystemDictionaryShared::find_or_allocate_info_for()` API to
>
> // Guaranteed to return non-NULL value for non-shared classes.
> // k must not be a shared class.
> DumpTimeClassInfo* SystemDictionaryShared::get_info(InstanceKlass* k)
>
>
> For more details, please see JBS issue: \
> [JDK-8290739](https://bugs.openjdk.org/browse/JDK-8290739)
This pull request has now been integrated.
Changeset: bc6a3c7b
Author: Ioi Lam <iklam@openjdk.org>
URL: https://git.openjdk.org/jdk/commit/bc6a3c7b833cb88b624f1d18877edc4f31102a48
Stats: 174 lines in 8 files changed: 26 ins; 55 del; 93 mod
8290739: Simplify storage of dump-time class information
Reviewed-by: ccheung
-------------
PR: https://git.openjdk.org/jdk/pull/9634
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic