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
>

Reply via email to