[lworld] RFR: 8247357: Flattenable field concept needs some cleanup

David Simms dsimms at openjdk.java.net
Fri Jun 12 08:09:00 UTC 2020


On Wed, 10 Jun 2020 18:46:33 GMT, Frederic Parain <fparain at openjdk.org> wrote:

> Please review these changes cleaning up the flattenable field concept.
> The concept has evolved with time and now all fields with an inline type are by definition flattenable, so the need to
> have a "flattenable bit" on the side is gone. The changeset contains a mix of renaming and code cleaning. The changes
> don't include JIT code, which would be fix in a follow-up patch.
> Thank you,
> 
> Fred

+1

> 
> Just wondering if "is_inlined()" wouldn't be an option as well? "Allocated" sounds like there is
> some heap allocation for that field going on.
> 
> Best regards,
> Tobias
>

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

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


More information about the valhalla-dev mailing list