RFR(XXS): 8222412: AARCH64: lse atomics encoding is not accepting zr as source

Dmitrij Pochepko dmitrij.pochepko at bell-sw.com
Mon Apr 15 10:20:22 UTC 2019


Thank you for review!

On 14/04/2019 5:28 PM, Dmitry Samersoff wrote:
> Dmitrij,
>
> The patch looks good to me.
>
> -Dmitry
>
> On 12.04.2019 18:24, Dmitrij Pochepko wrote:
>> Hi all,
>>
>> please review small fix for 8222412: AARCH64: lse atomics encoding is
>> not accepting zr as source
>>
>> webrev: http://cr.openjdk.java.net/~dpochepk/8222412/webrev.01/
>>
>> Current encoding for lse atomics hits assert when trying to use zr as
>> source register while it is allowed by spec. Current vm doesn't use
>> atomics with zr and this problem is not triggered.
>>
>>
>> Testing:
>>
>> I generated lse atomics with zr as source register. No assert observed
>> with patched vm.
>>
>>
>> CR: https://bugs.openjdk.java.net/browse/JDK-8222412
>>
>> Thanks,
>> Dmitrij
>>
>>


More information about the hotspot-compiler-dev mailing list