Thanks, Chris!
Serguei

On 5/30/13 8:41 AM, Christian Thalinger wrote:
Looks good.  -- Chris

On May 29, 2013, at 9:08 PM, [email protected] 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

Reply via email to