review(M): 7057120: Tiered: Allow C1 to inline methods with loops
Igor Veresov
igor.veresov at oracle.com
Wed Jun 22 17:12:48 PDT 2011
On 6/22/11 3:08 PM, Vladimir Kozlov wrote:
> Igor Veresov wrote:
>> Thanks for the review, Vladimir! Webrev updated. Please find the
>> replies inline.
>>
>> On 6/22/11 9:51 AM, Vladimir Kozlov wrote:
>>> Can you explain why in counter_overflow_helper() (c1_Runtime1.cpp) you
>>> expect the caller (enclosing_method) to be nmethod?
>>
>> It's not the caller. The enclosing_method is the actual method to
>> which the nmethod belongs. In other words: enclosing_method is a
>> method who's frame is current on stack and "method" is possibly the
>> inlinee inside the enclosing method that generated the event.
>
> It would be better to say this in the comment for
> counter_overflow_helper() method since the current comment does not help
> at all. I don't see
> anything related to safepoint.
Done, webrev updated.
igor
More information about the hotspot-compiler-dev
mailing list