[PATCH] D33719: Add _Float16 as a C/C++ source language type

[Bruno Cardoso Lopes via Phabricator via cfe-commits]

bruno added a comment.



> About the tests and using --target=aarch64: you're right that there should nothing be ARM specific here, but it is just that for Aarch64 it will show "half" IR types which I preferred, while it looks like x86 way of dealing with is to convert it and work on i16 types. Perhaps I need tests for both?

The ideal is to have x86 and ARM tests when testing for the codegen part of it. Keep in mind that you need "REQUIRES: <target>-registered-target" for those.


https://reviews.llvm.org/D33719