[PATCH] [AArch64 NEON] Support poly128_t and implement relevant intrinsic. -LLVM

Tim Northover t.p.northover at gmail.com
Fri Dec 6 02:38:32 PST 2013


> Also, I don't want bind i128 load/store to FPR128. In fact, NEON only
> provide 1 calculation(PMULL) working on i128. So I guess for most of the
> case loading or storing i128, the data will be used by some library
> functions running on cores instead of NEON, so storing i128 to two GPR64 is
> more general.

That's what I'd tend to think too. It's probably reasonable to
encourage the NEON intrinsic to generate a FPR128 load though. We
might well get that if we emitted:

    %tmp = load fp128* %addr
    %val = bitcast fp128 %tmp to i128

Sometimes opt might come in and mess up the load, but it should catch
a few cases that would otherwise cause extra register traffic.

Cheers.

Tim.



More information about the cfe-commits mailing list