review(M): 7057120: Tiered: Allow C1 to inline methods with loops
Vladimir Kozlov
vladimir.kozlov at oracle.com
Wed Jun 22 17:24:48 PDT 2011
Good.
Vladimir
Igor Veresov wrote:
> 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