Looks good to me Staffan!

(Those methods should have been called getXXXCount not Counter though :( )

David

On 8/05/2013 6:17 PM, Staffan Larsen wrote:
When JDK-8010862 (Method counter fields used for profiling can be allocated 
lazily) was fixed, SA was not updated to reflect this change.

The problem is with Method.getInvocationCounter() and 
Method.getBackedgeCounter() which do not handle the case where MethodCounters 
are not allocated.

The fix is to check for null and return 0 as the counter value.

webrev: http://cr.openjdk.java.net/~sla/8013591/webrev.00/
bug: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=8013591

Thanks,
/Staffan

Reply via email to