[lworld] RFR: 8249257 Rename ValueKlass to InlineKlass

Frederic Parain fparain at openjdk.java.net
Wed Jul 15 18:39:43 UTC 2020


On Wed, 15 Jul 2020 18:35:07 GMT, Frederic Parain <fparain at openjdk.org> wrote:

>> Please review this tedious change.  It renames class ValueKlass to InlineKlass and renames its fields and methods.  It
>> does not rename ValueArrayKlass.  That will be a future change.
>> Also, this change does not rename things defined in gc or jit source files.
>> 
>> The change was tested with tiers 1 and 2 on Mac, Windows, and Linux x64, and tiers 3-5 on Linux x64.
>> 
>> Thanks, Harold
>
> Marked as reviewed by fparain (Committer).

Harold,

Thank you for doing this tedious but necessary renaming job.
Looks good to me, just a few comments, mostly about fixing comments (no need for another review).

Regards,

Fred

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

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



More information about the valhalla-dev mailing list