Re: [PATCH][AArch64] Add crypto_pmull attribute

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

Re: [PATCH][AArch64] Add crypto_pmull attribute

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

Re: [PATCH][AArch64] Add crypto_pmull attribute

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

Re: [PATCH][AArch64] Add crypto_pmull attribute

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

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

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

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

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

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

Re: [PATCH][AArch64] Add crypto_pmull attribute

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

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