Re: RFR (XXL): 8223347: Integration of Vector API (Incubator): x86 backend changes

2020-05-01 Thread Vladimir Kozlov
n, Sandhya ; hotspot-compiler-...@openjdk.java.net; core-libs-dev@openjdk.java.net; hotspot-dev Subject: Re: RFR (XXL): 8223347: Integration of Vector API (Incubator): x86 backend changes Changes seems fine. Nice work. Why it is called "vector-unstable branch"? Thanks, Vladimir

RE: RFR (XXL): 8223347: Integration of Vector API (Incubator): x86 backend changes

2020-05-01 Thread Viswanathan, Sandhya
Kozlov Sent: Friday, May 01, 2020 5:32 PM To: Viswanathan, Sandhya ; hotspot-compiler-...@openjdk.java.net; core-libs-dev@openjdk.java.net; hotspot-dev Subject: Re: RFR (XXL): 8223347: Integration of Vector API (Incubator): x86 backend changes Changes seems fine. Nice work. Why it is called

Re: RFR (XXL): 8223347: Integration of Vector API (Incubator): x86 backend changes

2020-05-01 Thread Vladimir Kozlov
Changes seems fine. Nice work. Why it is called "vector-unstable branch"? Thanks, Vladimir K On 4/3/20 5:16 PM, Viswanathan, Sandhya wrote: Hi, Following up on review requests of API [0], Java implementation [1] and General Hotspot changes[3] for Vector API, here's a request for review of

RE: RFR (XXL): 8223347: Integration of Vector API (Incubator): x86 backend changes

2020-04-06 Thread Viswanathan, Sandhya
): 8223347: Integration of Vector API (Incubator): x86 backend changes Hi Sandhya I looked at the patch over the weekend. It looks good to me and a lot of work is involved. I have a question. Is this patch intended to panama/dev or mainline jdk? Nit: macroAssembler_x86.cpp has extra line at 115. Regards

Re: RFR (XXL): 8223347: Integration of Vector API (Incubator): x86 backend changes

2020-04-06 Thread Vivek Deshpande
Hi Sandhya I looked at the patch over the weekend. It looks good to me and a lot of work is involved. I have a question. Is this patch intended to panama/dev or mainline jdk? Nit: macroAssembler_x86.cpp has extra line at 115. Regards, Vivek OpenJDK id: vdeshpande On Fri, Apr 3, 2020 at 5:18 PM V

RFR (XXL): 8223347: Integration of Vector API (Incubator): x86 backend changes

2020-04-03 Thread Viswanathan, Sandhya
Hi, Following up on review requests of API [0], Java implementation [1] and General Hotspot changes[3] for Vector API, here's a request for review of x86 backend changes required for supporting the API: JEP: https://openjdk.java.net/jeps/338 JBS: https://bugs.openjdk.java.net/browse/JDK-822