RFR (S) 8232674: Fix build and rename ShenandoahBarrierSet::oop_load_from_native_barrier

Zhengyu Gu zgu at redhat.com
Mon Oct 21 00:19:21 UTC 2019


Thanks for fixing it, Aleksey

On 10/20/19 4:33 PM, Aleksey Shipilev wrote:
> P1 bug:
>    https://bugs.openjdk.java.net/browse/JDK-8232674
> 
> I believe this is caused by missing definition of this method:
>    oop oop_load_from_native_barrier(oop obj, narrowOop* load_addr);

This method should never been used. I thought I can get away with not 
implementing it (just like with gcc on Linux).

I think the method body should just be:
  ...
   ShouldNotReacheHere();
   return NULL;
  ...

-Zhengyu



> 
> The way out is to generify it, the same way as we do it for SBS::load_reference_barrier. I also took
> this opportunity to rename the method to match the other LRB flavor: now the
> ShenandoahRuntime::load_reference_barrier_native wrapper looks right.
> 
> Fix:
>    https://cr.openjdk.java.net/~shade/8232674/webrev.01/
> 
> Testing: Windows x86_64 build, Linux x86_64 build, hotspot_gc_shenandoah
> 



More information about the hotspot-gc-dev mailing list