On Mon, Mar 06, 2017 at 05:09:34AM +0000, 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, > Naveen > > 2017-03-06 Julian Brown <jul...@codesourcery.com> > Naveen H.S <naveen.hurugalaw...@cavium.com> > > * config/aarch64/aarch64-simd.md (aarch64_crypto_pmulldi) > (aarch64_crypto_pmullv2di): Change type attribute to crypto_pmull. > * config/aarch64/thunderx2t99.md (thunderx2t99_pmull): New > reservation. > * config/arm/cortex-a57.md (cortex_a57_neon_type): Add crypto_pmull to > attribute type list for neon_multiply.
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. I see: gcc/config/arm/cortex-a53.md gcc/config/arm/exynos-m1.md Which your patch doesn't touch, but which have modeling for neon_mul_d_long. Thanks, James > * config/arm/crypto.md (crypto_vmullp64): Change type to crypto_pmull. > * config/arm/types.md (crypto_pmull): Add. > * config/arm/xgene1.md (xgene1_neon_pmull): Add crypto_pmull to > attribute type list.
