Re: [PATCH][AArch64] Add crypto_pmull attribute

2017-06-20 Thread Hurugalawadi, Naveen
Hi Ramana, Thanks for the review and approval. >> Please update the ARM backend with the new attribute too >> (define_insn "crypto_vmullp64" Its already been updated in the patch posted at:- https://gcc.gnu.org/ml/gcc-patches/2017-03/msg00504.html >> Ok with that change and checking that you ca

Re: [PATCH][AArch64] Add crypto_pmull attribute

2017-06-20 Thread Ramana Radhakrishnan
On Tue, Jun 20, 2017 at 10:26 AM, Hurugalawadi, Naveen wrote: > Hi James, > > Thanks for the approval. > >>> From an AArch64 perspective, this is OK - but please wait for an ARM >>> approval before you commit it. > > Can anyone from ARM comment on the patch so that it can be committed > upstream i

Re: [PATCH][AArch64] Add crypto_pmull attribute

2017-06-20 Thread Hurugalawadi, Naveen
Hi James, Thanks for the approval. >> From an AArch64 perspective, this is OK - but please wait for an ARM >> approval before you commit it. Can anyone from ARM comment on the patch so that it can be committed upstream if no issues. https://gcc.gnu.org/ml/gcc-patches/2017-03/msg00504.html Than

Re: [PATCH][AArch64] Add crypto_pmull attribute

2017-06-06 Thread James Greenhalgh
On Fri, Mar 10, 2017 at 06:37:30AM +, Hurugalawadi, Naveen wrote: > Hi James, > > >> You need to do this for all cores which might be affected by this change, > >> i.e. all those which model neon_mul_d_long. > > Thanks for pointing out the missing cores in patch. > Added the support as per yo

Re: [PING 3][PATCH][AArch64] Add crypto_pmull attribute

2017-05-25 Thread Hurugalawadi, Naveen
Hi,   Please consider this as a personal reminder to review the patch at following link and let me know your comments on the same.   https://gcc.gnu.org/ml/gcc-patches/2017-03/msg00504.html Thanks, Naveen    

Re: [PING2][PATCH][AArch64] Add crypto_pmull attribute

2017-05-10 Thread Hurugalawadi, Naveen
Hi,   Please consider this as a personal reminder to review the patch at following link and let me know your comments on the same.   https://gcc.gnu.org/ml/gcc-patches/2017-03/msg00504.html Thanks, Naveen    

[PING][PATCH][AArch64] Add crypto_pmull attribute

2017-04-25 Thread Hurugalawadi, Naveen
Hi,   Please consider this as a personal reminder to review the patch at following link and let me know your comments on the same.   https://gcc.gnu.org/ml/gcc-patches/2017-03/msg00504.html Thanks, Naveen

Re: [PATCH][AArch64] Add crypto_pmull attribute

2017-03-09 Thread Hurugalawadi, Naveen
Hi James, >> You need to do this for all cores which might be affected by this change, >> i.e. all those which model neon_mul_d_long. Thanks for pointing out the missing cores in patch. Added the support as per your comments. Please find attached the modified patch and let us know if its okay fo

Re: [PATCH][AArch64] Add crypto_pmull attribute

2017-03-09 Thread James Greenhalgh
On Mon, Mar 06, 2017 at 05:09:34AM +, Hurugalawadi, Naveen wrote: > Hi, > > Please find attached the patch that adds "crypto_pmull" for AArch64. > > Bootstrapped and Regression tested on aarch64-thunder-linux. > > Please review the patch and let us know if its okay for Stage-1? > > Thanks,

[PATCH][AArch64] Add crypto_pmull attribute

2017-03-05 Thread Hurugalawadi, Naveen
Hi, Please find attached the patch that adds "crypto_pmull" for AArch64. Bootstrapped and Regression tested on aarch64-thunder-linux. Please review the patch and let us know if its okay for Stage-1? Thanks, Naveen 2017-03-06 Julian Brown Naveen H.S * config/aarch64/aa