On 10/30/12 10:37 AM, Daniel D. Daugherty wrote:
Thumbs up.

Thanks, Dan!
Serguei



Is someone going to do an audit for similar missing changes
from max_stack() (not max_size()) to verifier_max_stack()?

Dan


On 10/30/12 1:30 PM, serguei.spit...@oracle.com wrote:
Hello,


Please, review the fix for CR:
http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7194607

CR in JIRA:
https://jbs.oracle.com/bugs/browse/JDK-7194607

Open webrev:
http://cr.openjdk.java.net/~sspitsyn/webrevs/2012/7194607-JVMTI-max_size


Summary:

This issue is caused by the changes in the oops/method.hpp for invokedynamic (JSR 292). Now the max_stack() adds +2 to the original code attribute stack size if invokedynamic is enabled. The verifier_max_stack() must be used in the jvmtiClassFileReconstituter.cpp
instead of the max_size() to get the code attribute stack size.


Thanks,
Serguei


Reply via email to