RFR: 8135157: DMB elimination in AArch64 C2 synchronization implementation

Vladimir Kozlov vladimir.kozlov at oracle.com
Fri Sep 11 04:24:11 UTC 2015


Very nice cleanup! Looks good.

Thanks,
Vladimir

On 9/9/15 1:16 AM, Edward Nevill wrote:
> On Tue, 2015-09-08 at 16:20 +0100, Andrew Haley wrote:
>> On 09/08/2015 09:49 AM, Andrew Haley wrote:
>>> On 09/08/2015 09:36 AM, Edward Nevill wrote:
>>>>>
>>>> -  ins_encode(aarch64_enc_cmpxchg(mem, oldval, newval));
>>>> +  ins_encode(aarch64_enc_cmpxchg_acq(newval, heap_top_ptr));
>>>>
>>>> I think the original line was correct in this case.
>>>
>> http://cr.openjdk.java.net/~aph/8135157-2/
>
> This rev looks good.
>
> Tested OK jcstress on 3 different partner HW.
>
> All the best,
> Ed.
>
>


More information about the hotspot-compiler-dev mailing list