On Tue, Jun 20, 2017 at 10:26 AM, Hurugalawadi, Naveen <naveen.hurugalaw...@cavium.com> 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 if no issues. > > https://gcc.gnu.org/ml/gcc-patches/2017-03/msg00504.html
Please update the ARM backend with the new attribute too .... crypto.md: (define_insn "crypto_vmullp64" [(set (match_operand:TI 0 "register_operand" "=w") (unspec:TI [(match_operand:DI 1 "register_operand" "w") (match_operand:DI 2 "register_operand" "w")] UNSPEC_VMULLP64))] "TARGET_CRYPTO" "vmull.p64\\t%q0, %P1, %P2" [(set_attr "type" "neon_mul_d_long")] ) Ok with that change and checking that you can build cc1 for arm-none-eabi . regards Ramana > > Thanks, > Naveen >