[lworld] RFR: 8253511: [lworld] ClassFileParser does not handle field type mismatch [v2]

Harold Seigel hseigel at openjdk.java.net
Wed Sep 23 17:39:35 UTC 2020


On Wed, 23 Sep 2020 17:36:17 GMT, Frederic Parain <fparain at openjdk.org> wrote:

>> Please review this change with replaces an assert with the throwing of an IncompatibleClassChangeError when a
>> pre-loaded class is not an inline type. This case was correctly handled for static fields, but not for non-static
>> fields.  Tested with Mach5, tiers 1 to 3.
>> 
>> Thank you,
>> 
>> Fred
>
> Frederic Parain has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Verify ICE message in unit test

Looks good!  Thanks!

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

Marked as reviewed by hseigel (Committer).

PR: https://git.openjdk.java.net/valhalla/pull/195


More information about the valhalla-dev mailing list