review(M): 7057120: Tiered: Allow C1 to inline methods with loops
Tom Rodriguez
tom.rodriguez at oracle.com
Thu Jun 23 10:13:16 PDT 2011
Looks good.
tom
On Jun 22, 2011, at 5:34 PM, Igor Veresov wrote:
> Thanks, Vladimir!
>
> igor
>
> On 6/22/11 5:24 PM, Vladimir Kozlov wrote:
>> 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