[lworld] RFR: 8248003: [lworld] [lw3] VM crashes when classes with inline type fields are loaded from CDS archive

Ioi Lam ioi.lam at oracle.com
Wed Jun 24 00:49:58 UTC 2020


Hi Fred,

The changes look good to me. For this comment:

// Even if the Point class is not loaded from the CDS archive, make sure 
the WithInlineField class
// can still be loaded successfully, and properly get the rewritten 
version of Point.

[1] spelling of WithInlineField -> WithInlinedField
[2] I think you can add

// The archived version of WithInlinedField must not be loaded, because 
it references the archived
// version of Point, but a different version of Point has been loaded.


Thanks
- Ioi

On 6/23/20 10:52 AM, Frederic Parain wrote:
> On Mon, 22 Jun 2020 12:46:56 GMT, Frederic Parain <fparain at openjdk.org> wrote:
>
>> Please review this changeset fixing several issues in CDS related to inline type metadata removal and restoration.
>>
>> Thank you,
>>
>> Fred
> The fix has been updated according to Ioi's review, and a new test has been added with class redefinition.
>
> Thank you,
>
> Fred
>
> -------------
>
> PR: https://git.openjdk.java.net/valhalla/pull/95



More information about the valhalla-dev mailing list