commit 2700809c6a1badc0333bdc0d3957847774e44b1a
author Marat Dukhan <maratek@google.com> Wed Dec 29 11:30:06 2021 -0800
committer XNNPACK Team <xnnpack-github-robot@google.com> Wed Dec 29 11:31:06 2021 -0800
tree 6da5e1a570a0da7d10d6cb28170322e0648ed174
parent 70137e45015534d8fde167a2d2bd18d593daaf4b

Specify -mfp16-format=ieee for AArch32 GCC builds

PiperOrigin-RevId: 418823088

CMakeLists.txt

diff --git a/CMakeLists.txt b/CMakeLists.txt
index bc78cbc..5840923 100755
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -5549,6 +5549,10 @@
   SET_PROPERTY(SOURCE ${ALL_MICROKERNEL_SRCS} APPEND_STRING PROPERTY COMPILE_FLAGS " -marm ")
   SET_PROPERTY(SOURCE ${ALL_NEON_MICROKERNEL_SRCS} APPEND_STRING PROPERTY COMPILE_FLAGS " -march=armv7-a -mfpu=neon ")
   SET_PROPERTY(SOURCE ${ALL_NEONFP16_MICROKERNEL_SRCS} APPEND_STRING PROPERTY COMPILE_FLAGS " -march=armv7-a -mfpu=neon-fp16 ")
+  # GCC requires -mfp16-format=ieee to define __fp16 type, but Clang doesn't support this option at all.
+  IF(CMAKE_C_COMPILER_ID STREQUAL "GNU")
+    SET_PROPERTY(SOURCE ${ALL_NEONFP16_MICROKERNEL_SRCS} APPEND_STRING PROPERTY COMPILE_FLAGS " -mfp16-format=ieee ")
+  ENDIF()
   SET_PROPERTY(SOURCE ${ALL_NEONFMA_MICROKERNEL_SRCS} APPEND_STRING PROPERTY COMPILE_FLAGS " -march=armv7-a -mfpu=neon-vfpv4 ")
   IF(IOS)
     SET_PROPERTY(SOURCE ${ALL_NEONV8_MICROKERNEL_SRCS} APPEND_STRING PROPERTY COMPILE_FLAGS " -mcpu=cyclone -mtune=generic ")