RFR: 8199739: Use HeapAccess when loading oops from static fields in javaClasses.cpp

Stefan Karlsson stefan.karlsson at oracle.com
Mon Mar 19 10:11:25 UTC 2018


Hi all,

Kim and Roman commented that my patch doesn't work with Shenandoah. 
Here's an updated version:
http://cr.openjdk.java.net/~stefank/8199739/webrev.02/

Thanks,
StefanK

On 2018-03-16 15:39, Stefan Karlsson wrote:
> Hi all,
> 
> Please review this patch to use HeapAccess<>::oop_load instead of 
> oopDesc::load_decode_heap_oop when loading oops from static fields in 
> javaClasses.cpp:
> 
> http://cr.openjdk.java.net/~stefank/8199739/webrev.01/
> https://bugs.openjdk.java.net/browse/JDK-8199739
> 
> It's necessary to use HeapAccess<>::oop_load to inject load barriers for 
> GCs that need them.
> 
> Thanks,
> StefanK


More information about the hotspot-dev mailing list