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

Dmitry Samersoff dms at samersoff.net
Sun Apr 14 14:28:08 UTC 2019


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