RFR (S) 8197844: JVMTI GetLoadedClasses should use the Access API

[coleen.phillimore at oracle.com]

Thank you for noticing that.  I'll remove it.
Coleen

On 3/15/18 9:32 AM, Poonam Parhar wrote:
> Hi Coleen,
>
> Fix for JDK-8187577 <https://bugs.openjdk.java.net/browse/JDK-8187577> 
> added the following include too in jvmtiGetLoadedClasses.cpp file. I 
> think that can also be removed now.
>
> +#if INCLUDE_ALL_GCS
> +#include "gc/g1/g1SATBCardTableModRefBS.hpp"
> +#endif
>
> Thanks,
> Poonam
>
> On 3/14/2018 10:37 AM, coleen.phillimore at oracle.com wrote:
>> Summary: Make sure the holder of a class loader is accessed during 
>> iteration of CLDG
>>
>> This is where we should have put the GC barrier.  This can be cleaned 
>> somewhat when we have a WeakHandle holder in the ClassLoaderData, 
>> then the code in ensure_loader_alive() becomes _holder.resolve();
>>
>> Tested with tier1-5.
>>
>> open webrev at http://cr.openjdk.java.net/~coleenp/8197844.01/webrev
>> bug link https://bugs.openjdk.java.net/browse/JDK-8197844
>>
>> Thanks,
>> Coleen
>