[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