RFR(XS): 8230434: [C1, C2] Release barrier for volatile field stores in constructors implemented inconsistently

Schmidt, Lutz lutz.schmidt at sap.com
Wed Sep 4 10:07:39 UTC 2019


Hi Martin, 
the changes look good to me. I like the cleanup and the "synchronization" of coding style.
Please note that I'm NOT a Reviewer.
Thanks,
Lutz

On 03.09.19, 15:59, "hotspot-compiler-dev on behalf of Aleksey Shipilev" <hotspot-compiler-dev-bounces at openjdk.java.net on behalf of shade at redhat.com> wrote:

    On 9/3/19 3:37 PM, Doerr, Martin wrote:
    > Sure, I can also clean up coding style:
    > http://cr.openjdk.java.net/~mdoerr/8230434_membar_volatile_field_constructor/webrev.01/
    This looks good to me.
    
    -- 
    Thanks,
    -Aleksey
    
    



More information about the hotspot-compiler-dev mailing list