RFR (XXS) 8202164: Remove some unneeded BoolObjectClosure* is_alive parameters

[coleen.phillimore at oracle.com]

Thanks for the code review.
Coleen

On 4/24/18 1:39 AM, Stefan Karlsson wrote:
> Looks good (with Kim's suggestions)
>
> StefanK
>
> On 2018-04-24 03:28, Kim Barrett wrote:
>>> On Apr 23, 2018, at 7:12 PM, coleen.phillimore at oracle.com wrote:
>>>
>>> See bug for details.
>>>
>>> open webrev at http://cr.openjdk.java.net/~coleenp/8202164.01/webrev
>>> bug link https://bugs.openjdk.java.net/browse/JDK-8202164
>>>
>>> Tested with runThese (lots of class unloading) for all the GCs and 
>>> with mach5 hs-tier1 and 2.
>>> Thanks,
>>> Coleen
>> I don’t think I’d call this XXS.
>>
>> Looks good.
>>
>> Needs a couple comment updates; I don't need a new webrev for these.
>>
>> ------------------------------------------------------------------------------ 
>>
>> src/hotspot/share/code/nmethod.cpp
>> 1039   flush_dependencies(/*delete_immediately*/false);
>>
>> The preceding comment needs to be updated; there's no longer a closure
>> involved.
>>
>> ------------------------------------------------------------------------------ 
>>
>> src/hotspot/share/code/nmethod.cpp
>> 1364         // During GC delete_immediately is false, and is used to
>> 1365         // determine liveness of dependees that need to be updated.
>> 1366         if (delete_immediately || klass->is_loader_alive()) {
>>
>> The comment needs to be updated; delete_immediately doesn't determine
>> liveness.
>>
>> ------------------------------------------------------------------------------ 
>>
>>
>