RFR: 8035393: Use CLDClosure instead of CLDToOopClosure in frame::oops_interpreted_do
Stefan Karlsson
stefan.karlsson at oracle.com
Thu Feb 20 12:16:10 PST 2014
On 2014-02-20 21:04, Coleen Phillimore wrote:
>
> Looks good. CLDClosure is a shorter name too.
Thanks, Coleen.
StefanK
> Coleen
>
> On 2/20/14 9:28 AM, Stefan Karlsson wrote:
>>
>> On 2014-02-20 12:35, Mikael Gerdin wrote:
>>> Stefan,
>>>
>>> On Thursday 20 February 2014 10.59.41 Stefan Karlsson wrote:
>>>> Hi all,
>>>>
>>>> Please, review this patch to change the closure type used in
>>>> frame::oops_interpreted_do from CLDToOopClosure to CLDClosure,
>>>> which is
>>>> an abstract base class of CLDToOopClosure.
>>>>
>>>> We need this for the G1 Class Unloading project, to be able to
>>>> specialize the CLD scanning code during the Initial Mark phase.
>>>>
>>>> http://cr.openjdk.java.net/~stefank/8035393/webrev.00/
>>> Looks good.
>>
>> Thanks for the review.
>>
>> StefanK
>>
>>> /Mikael
>>>
>>>> thanks,
>>>> StefanK
>>
>
More information about the hotspot-dev
mailing list