Looks good. -- Chris On May 29, 2013, at 9:08 PM, serguei.spit...@oracle.com wrote:
> Please, review the fix and unit test for: > bug: http://bugs.sun.com/view_bug.do?bug_id=8015436 > jbs: https://jbs.oracle.com/bugs/browse/JDK-8015436 > > Open webrev: > http://cr.openjdk.java.net/~sspitsyn/webrevs/2013/hotspot/8015436-JVMTI-JSR292.1 > > Summary: > The InstanceKlass _initial_method_idnum value must be adjusted as necessary > if the overpass methods are added. > I guess, it is Ok to integrate a unit test covering the fix under the same > bug id. > > Testing: > Newly added hotspot unit test: compiler/8015436/Test8015436.java > The originally failed test: compiler/ciReplay/TestSA.sh > The vm/mlvm tests > > Thanks, > Serguei
