Initial runtime support for the ValueTypes attribute
Karen Kinnear
karen.kinnear at oracle.com
Fri May 18 14:16:05 UTC 2018
Frederic,
Code looks really good. Many thanks for doing this so quickly and carefully.
Thank you for the symbol refcount handling.
Summary:
declared fields (static and instance): preload, i.e. classfileparser loads before completing container loading
linkage to fields and methods, we load at link time prior to creation of vtables/itables
- this all sounds correct to me.
instanceKlass.cpp: 621
- comment is that arrays of value types are not handled
- do we actually need to preload arrays of value types at link time for any optimizations?
Empty.java: line 40: Excepted -> Expected
thanks,
Karen
> On May 17, 2018, at 4:04 PM, Frederic Parain <frederic.parain at oracle.com> wrote:
>
> Please review this first patch related to the ValueTypes attribute:
>
> http://cr.openjdk.java.net/~fparain/VTAttribute/webrev.00/
>
> The patch includes:
> - the parsing of the ValueTypes attribute
> - the creation of meta-data from this attribute
> - a consistency check between the ACC_FLATTENABLE flag
> and the ValueTypes attribute
> - the pre-loading of method arguments types and return values types
>
> Fred
>
More information about the valhalla-dev
mailing list