[13] RFR(L) 8220623: [JVMCI] Update JVMCI to support JVMCI based Compiler compiled into shared library

Vladimir Kozlov vladimir.kozlov at oracle.com
Wed Apr 3 01:39:41 UTC 2019


On 4/2/19 5:49 PM, Kim Barrett wrote:
>> On Apr 2, 2019, at 7:51 PM, Kim Barrett <kim.barrett at oracle.com> wrote:
>> src/hotspot/share/runtime/jniHandles.cpp
>> 192 #if INCLUDE_JVMCI
>> 193   JVMCI::oops_do(f);
>> 194 #endif
>>
>> I don't think that belongs here.
> 
> In addition to my thinking this is just out of place (we already removed the similar JVMTI
> piggybacking from JNIHandles::weak_oops_do, though it had other problems too), note
> that ZGC doesn’t call JNIHandles::oops_do at all (it uses the parallel oopstorage API), and
> probably the only reason some of the other collectors (particularly G) still do is because
> nobody has gotten around to making them use the parallel oopstorage API too.
> 

This is ported from JDK 8 changes. Please, give me suggestion where I should call it?

Thanks,
Vladimir


More information about the graal-dev mailing list