[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