[lworld] RFR: 8281116: [lworld] Adding Preload attribute support
Harold Seigel
hseigel at openjdk.java.net
Wed Feb 2 14:25:42 UTC 2022
On Wed, 2 Feb 2022 13:12:19 GMT, Frederic Parain <fparain at openjdk.org> wrote:
> Please review this changeset adding support for the Preload attribute.
> It includes a logging feature to track and diagnose those optimistic class loading.
>
> Tested with Mach5 tier1.
>
> Thank you,
>
> Fred
src/hotspot/share/classfile/classFileParser.cpp line 5951:
> 5949: _record_components(NULL),
> 5950: _temp_local_interfaces(NULL),
> 5951: _local_interfaces(NULL),
Do you need to free _preload_classes in ClassFileParser::~ClassFileParser() and set _preload_classes to NULL in ClassFileParser::clear_class_metadata(), also, assert that it is NULL in ClassFileParser::fill_instance_klass() ?
-------------
PR: https://git.openjdk.java.net/valhalla/pull/623
More information about the valhalla-dev
mailing list