commit c4402620a2d382793c1ddf1662d1d7fa9a260e6c
author Siva Chandra <sivachandra@google.com> Wed Jan 27 10:29:30 2021 -0800
committer Copybara-Service <copybara-worker@google.com> Thu Jan 28 12:33:19 2021 -0800
tree 352502a698756964088f7bf05a0a971d92d10c65
parent 51a5a1161d8997a9b17a10b90cf00b5ab4fe1ebc

[libc] Disable sqrtl_test on non-x86 platforms.

The added comment explains why it has been disabled for now. We will
enable back when we fix the problem.

GitOrigin-RevId: 74c87a363fd0e8586e46c04e113c49c6dc2a0bc8

test/src/math/CMakeLists.txt

diff --git a/test/src/math/CMakeLists.txt b/test/src/math/CMakeLists.txt
index 48f8902..e0c45b8 100644
--- a/test/src/math/CMakeLists.txt
+++ b/test/src/math/CMakeLists.txt
@@ -924,18 +924,26 @@
     libc.utils.FPUtil.fputil
 )
 
-add_fp_unittest(
-  sqrtl_test
-  NEED_MPFR
-  SUITE
-    libc_math_unittests
-  SRCS
-    sqrtl_test.cpp
-  DEPENDS
-    libc.include.math
-    libc.src.math.sqrtl
-    libc.utils.FPUtil.fputil
-)
+# The quad precision test for sqrt against MPFR currently suffers
+# from insufficient precision in MPFR calculations leading to
+# https://hal.archives-ouvertes.fr/hal-01091186/document. We will
+# renable after fixing the precision issue.
+if(${LIBC_TARGET_MACHINE} MATCHES "x86_64")
+  add_fp_unittest(
+    sqrtl_test
+    NEED_MPFR
+    SUITE
+      libc_math_unittests
+    SRCS
+      sqrtl_test.cpp
+    DEPENDS
+      libc.include.math
+      libc.src.math.sqrtl
+      libc.utils.FPUtil.fputil
+  )
+else()
+  message(STATUS "Skipping sqrtl_test")
+endif()
 
 add_fp_unittest(
   remquof_test