On 16 January 2015 at 18:22, Alan Lawrence <alan.lawre...@arm.com> wrote: > These add all the V[48]HFmode insns and corresponding intrinsics for ARM. > Depends on the two patches at > https://gcc.gnu.org/ml/gcc-patches/2015-01/msg01422.html . > > Unfortunately I don't at present have a testsuite. I've done some testing > both manually and on a large internal testsuite for Neon/ACLE intrinsics, > but I'm wondering if anyone has anything they might be able to contribute? > Christophe, perhaps you can give me some pointers, how might one add float16 > to the advsimd-intrinsics testsuite / how easy would this be?
I don't expect this to be difficult. I have some support already in my testsuite at https://gitorious.org/arm-neon-tests For instance, if you look at ref_vld1.c, you'll that GCC's vld1.c test needs to have something like: #if defined(__ARM_FP16_FORMAT_IEEE) DECL_VARIABLE(vector, float, 16, 4); DECL_VARIABLE(vector, float, 16, 8); #endif ... #if defined(__ARM_FP16_FORMAT_IEEE) TEST_VLD1(vector, buffer, , float, f, 16, 4); TEST_VLD1(vector, buffer, q, float, f, 16, 8); #endif as well as a similar fragment with the expected results. I didn't add it yet because I was waiting for the support in GCC to be complete :-) I discussed this briefly with Ramana when I started contributing my tests. However, it might be better to add such tests in a dedicated file (e.g. vld1-fp16.c) protected by some dejagnu directive to check that the effective target does support fp16, and without the #ifdef. > Cross-tested check-gcc on arm-none-eabi > Bootstrapped on arm-none-linux-gnueabihf cortex-a15 > Christophe.
