RFR: Verify liveness data
Aleksey Shipilev
shade at redhat.com
Mon Aug 7 15:37:09 UTC 2017
On 08/07/2017 03:49 PM, Zhengyu Gu wrote:
> 1) ShenandoahLivenessData* _ld; -> volatile ShenandoahLivenessData* _ld;
>
> 2) shenandoahVerifier.cpp
> 251 Atomic::add(obj->size() + BrooksPointer::word_size(), _ld + obj_reg->region_number());
>
> -> 251 Atomic::add(obj->size() + BrooksPointer::word_size(),
> _ld[obj_reg->region_number()]);
Fixed both.
It seems to me that volatility is required in ShenandoahLivenessData definition itself:
http://cr.openjdk.java.net/~shade/shenandoah/verifier-liveness/webrev.02/
Thanks,
-Aleksey
More information about the shenandoah-dev
mailing list