RFR: 8009719: core reflection should get type annotation data from the VM lazily

[Erik Joelsson]

I'm happy with the build part now.

/Erik

On 2013-09-23 16:00, Joel Borggren-Franck wrote:
> Hi all,
>
> Updated webrev: http://cr.openjdk.java.net/~jfranck/8009719/webrev.02/
>
> Adds Field.c to make/java/java/FILES_c.gmk (old build)
> Renames parameter in Field.c from method to field
>
> Thanks for the suggestions/fixes!
>
> cheers
> /Joel
>
> On 2013-09-21, Joel Borggren-Franck wrote:
>> Hi
>>
>> A while ago [1] I introduced an extra field in to j.l.r.Method,
>> j.l.r.Constructor, and j.l.r.Field in order to support reflection for
>> type annotations. These fields were intended to be removed later, they
>> were there to make the coordination between VM and libraries easier when
>> implementing reflection for type annotations.
>>
>> This change removes the fields. Reflection for type annotations simply
>> get the bytes from the vm lazily when it needs them. Because reflection
>> for type annotations is suspected to be fairly uncommon and not
>> performance critical no caching is done. This can be changed later.
>>
>> The vm side of things were pushed a while back [2], this is the JDK side
>> of the changes.
>>
>> This is a refactoring, all current annotation and type annotation tests
>> pass after this change.
>>
>> Webrev: http://cr.openjdk.java.net/~jfranck/8009719/webrev.01/
>> Bug id: https://bugs.openjdk.java.net/browse/JDK-8009719
>>
>> Also including build-dev since I needed to update the mapfiles. To my
>> knowledge I have updated both the old and the new build.
>>
>> Please review
>>
>> cheers
>> /Joel
>>
>> [1] : http://hg.openjdk.java.net/jdk8/jdk8/jdk/rev/6d977f61af5e
>> [2] : http://hg.openjdk.java.net/jdk8/jdk8/hotspot/rev/1916ca1dec2f