RFR: 8158854: Ensure release_store is paired with load_acquire in lock-free code
Aleksey Shipilev
aleksey.shipilev at gmail.com
Thu Aug 18 14:15:01 UTC 2016
On 08/18/2016 05:50 AM, David Holmes wrote:
> webrev: http://cr.openjdk.java.net/~dholmes/8158854/webrev/
Looks good to me. Minor nit:
*) inline declarations have different indenting, is that our code style?
370 Klass* array_klasses() const { return _array_klasses; }
371 inline Klass* array_klasses_acquire() const; // load with
acquire semantics
372 void set_array_klasses(Klass* k) { _array_klasses = k; }
373 inline void release_set_array_klasses(Klass* k); // store with
release semantics
Thanks,
-Aleksey
More information about the hotspot-runtime-dev
mailing list