RFR: Remove obsolete and unused reduce-storeval-barrier optimization code

Aleksey Shipilev shade at redhat.com
Mon Sep 18 09:58:01 UTC 2017


On 09/16/2017 11:37 AM, Roman Kennke wrote:
> Ok. I fixed one place in library_call.cpp in the cmpxchg intrinsics. Everything else should be ok:
> 
> http://cr.openjdk.java.net/~rkennke/remove-reduce-storeval/webrev.01/

*) I wonder if instead of putting shenandoah_storeval_barrier prior to store_oop_* calls, we are
better putting the barriers calls inside of them? Under UseShenandoahGC flag? Otherwise, I see
missing shenandoah_storeval_barrier in GraphKit::builtin_throw and Parse::expand_multianewarray.

Seems good otherwise.

Thanks,
-Aleksey



More information about the shenandoah-dev mailing list