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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
