The aarch64 ARMv8.2-A FP16 vector intrinsic patch (https://reviews.llvm.org/D32511) is reverted because it broke some tests for arm. Work is being done to add fp16 types for arm backend (https://reviews.llvm.org/D38315). While waiting for that work to be finished, the code in this patch guards builtin native fp16 data type so that it can only be used by aarch64.
Details
Details
Diff Detail
Diff Detail
Event Timeline
Comment Actions
Looks reasonable to me.
Just checking, do we need those changes in CodeGen/arm_neon_intrinsics.c? I mean, trunk is already using i16 in these tests.
Comment Actions
It depends on how I check in the old reverted code (the one in https://reviews.llvm.org/D32511) with this one. If I put back the reverted code as is first and then check in this one separately, then we need the changes in arm_neon_intrinsics.c. If we combine this one with the old one into one checkin, then it is not needed. I will probably go with the former so that when support for fp16 in arm32 in complete, we can revert the whole code here but I am open to suggestions.