[vector] RFR 8226918: Fix Vector API single jtreg failures for AArch64 NEON

[Ningsheng Jian]

Hi Yang,

Your fix looks good to me. It seems that x86 has the same issue?

With the fix, dst will also be a different reg from src. This looks like 
a limitation of current RA.

Thanks,
Ningsheng

On 8/5/19 11:00 AM, Yang Zhang (Arm Technology China) wrote:
> Hi
> 
> Ping it again. Could anyone please help to review this patch?
> 
> Regards
> Yang
> -----Original Message-----
> From: Yang Zhang (Arm Technology China)
> Sent: Friday, June 28, 2019 3:34 PM
> To: panama-dev at openjdk.java.net
> Cc: nd <nd at arm.com>
> Subject: [vector] RFR 8226918: Fix Vector API single jtreg failures for AArch64 NEON
> 
> Hi
> 
> Could anyone please help to review this patch?
> 
> JBS: https://bugs.openjdk.java.net/browse/JDK-8226918
> Webrev: http://cr.openjdk.java.net/~yzhang/vectorapi/8226918/webrev.00/
> 
> In float insert instructions, vRegF is low 32-bit of NEON register. Without TEMP_DEF, dst might use the same register with val. In such case, val may be overwritten by moving src to dst.
> With this patch, single jtreg failures are fixed, and no new failure.
> 
> Regards,
> Yang
>