[PATCH] D44538: [ARM] Support for v4f16 and v8f16 vectors

[Sjoerd Meijer via Phabricator via llvm-commits]

SjoerdMeijer added a comment.

I have uploaded (companion) Clang patch: https://reviews.llvm.org/D44561
this passes the Half Type when it is appropriate to do so, and i16 otherwise.
Thus, the normal neon intrinsics work as before and as expected, and with this
LLVM patch for f16 vectors, the FP16 vector intrinsics are fine too.

About the bitconverts, they were actually necessary to get the code generation
working for the 2 intrinsics in the regression tests.  They run in hard and
softfp mode, and these bitconverts patterns were necessary for the softfp case,
but I will double check to be sure.

If we are happy with this, then I will first add (or actually reenable) the AArch32
FP16 vector intrinsics to clang again, and then complete the rest of the LLVM
codegen tests.


https://reviews.llvm.org/D44538