Hi Karen,
On 10/10/2013 8:13 AM, Karen Kinnear wrote:
Please review:
webrev: http://cr.openjdk.java.net/~acorn/8026213/webrev/
bug: https://bugs.openjdk.java.net/browse/JDK-8026213
Summary:
Reflection generates code dynamically to speed up reflection processing after
startup. The first
15 runs of a reflection call use the vm code path, after that we use the
generated code path, which
needs to use invokespecial on private methods in interfaces.
Minor nit:
30 import java.lang.reflect.*;
The approved style these days is to list the exact imports needed rather
than rely on import-on-demand.
Otherwise change seems fine. Is the VM side of this already in place or
does this need to sync with a VM change?
Thanks,
David
Tested:
Test attached to the bug
Also - all the 8011311 private method testing was run with this in the build:
Robert Field's TypeTest
8025475 test
defmeth privatemethodstest with reflection
John Rose's intfbug
jtreg: java.util, java.lang
jck vm, lang
thanks,
Karen
