[PATCH] D43650: [ARM] Add ARMv8.2-A FP16 vector intrinsics

Sjoerd Meijer via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Mar 6 06:39:57 PST 2018


SjoerdMeijer accepted this revision.
SjoerdMeijer added a comment.
This revision is now accepted and ready to land.

Look like sensible cleanups/fixes/additions to me.

We were struggling whether to pass an i16 or f16 type, which can both be illegal types.
Therefore, it perhaps doesn't really matter much which one we use. At the time we
were having these problems, the ARM backend wasn't ready for f16s, but a lot has changed
since then. So I think we can get this in, review where we are, and iterate on this
if necessary.


https://reviews.llvm.org/D43650





More information about the cfe-commits mailing list