RFR(XS) 8224558: x86 Fix replicateB encoding

[Deshpande, Vivek R]

Thanks. Pushed the patch.
http://hg.openjdk.java.net/jdk/jdk/rev/d1fa0f8d8c9a

Regards,
Vivek
-----Original Message-----
From: Vladimir Ivanov [mailto:vladimir.x.ivanov at oracle.com] 
Sent: Monday, May 27, 2019 4:14 AM
To: Deshpande, Vivek R <vivek.r.deshpande at intel.com>; 'hotspot-compiler-dev at openjdk.java.net compiler' <hotspot-compiler-dev at openjdk.java.net>
Cc: Viswanathan, Sandhya <sandhya.viswanathan at intel.com>
Subject: Re: RFR(XS) 8224558: x86 Fix replicateB encoding


> http://cr.openjdk.java.net/~vdeshpande/8224558/webrev.01/

Looks good.

Best regards,
Vladimir Ivanov

> I can push the patch, after both of you find it ok.
> 
> Regards,
> Vivek
> 
> -----Original Message-----
> From: Tobias Hartmann [mailto:tobias.hartmann at oracle.com]
> Sent: Thursday, May 23, 2019 5:45 AM
> To: Deshpande, Vivek R <vivek.r.deshpande at intel.com>; 'hotspot-compiler-dev at openjdk.java.net compiler' <hotspot-compiler-dev at openjdk.java.net>
> Cc: Viswanathan, Sandhya <sandhya.viswanathan at intel.com>
> Subject: Re: RFR(XS) 8224558: x86 Fix replicateB encoding
> 
> Hi Vivek,
> 
> On 22.05.19 19:48, Deshpande, Vivek R wrote:
>> I came across this issue with vector API tests.
>> I tried to write a reproducer using the autovectorizer test, but it always uses the register based rule(instruct Repl32B) instead of memory based rule (instruct Repl32B_mem) and register based rule gives correct result.
>> This is why it never showed up.
> 
> Okay, thanks for the explanation.
> 
>> Could you please help me with forcing to use memory based rule with autovectorizer based test.
> I'm afraid I can't help, I'm not even sure what the autovectorizer is. Is that specific to the vector API?
> 
> Best regards,
> Tobias
>