about summary refs log tree commit diff
diff options
context:
space:
mode:
authorChris Metcalf <cmetcalf@ezchip.com>2015-07-10 16:11:40 -0400
committerChris Metcalf <cmetcalf@ezchip.com>2015-07-10 16:11:40 -0400
commit75413d4928d6b3cf4602e9251855b3fc53730abd (patch)
tree5561306b6a5d07959f32331f9ae65436fd0c3e9d
parent0c8f39cbf769ef35f54b57859e1e87b5291daf5e (diff)
downloadglibc-75413d4928d6b3cf4602e9251855b3fc53730abd.tar.gz
glibc-75413d4928d6b3cf4602e9251855b3fc53730abd.tar.xz
glibc-75413d4928d6b3cf4602e9251855b3fc53730abd.zip
math/test-fenvinline: avoid compiler warning
On tile (and any other machine with no FP exceptions) the
feenable_test() function will generate a "function defined but
not used" warning because all of the callers are commented out.
We already were ifdef'ing out the body of the function, so instead
just ifdef out the entire function if FE_ALL_EXCEPT == 0.
-rw-r--r--ChangeLog5
-rw-r--r--math/test-fenvinline.c4
2 files changed, 7 insertions, 2 deletions
diff --git a/ChangeLog b/ChangeLog
index 0c7fc2ca92..8188ee3a20 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@
+2015-07-10  Chris Metcalf  <cmetcalf@ezchip.com>
+
+	* math/test-fenvinline.c (feenable_test) [FE_ALL_EXCEPT == 0]:
+	Comment out the whole function, not just its body.
+
 2015-07-10  Siddhesh Poyarekar  <siddhesh@redhat.com>
 
 	* stdlib/tst-tls-atexit.c (load): Remove unnecessary mutex
diff --git a/math/test-fenvinline.c b/math/test-fenvinline.c
index b02cefd7f8..d196268045 100644
--- a/math/test-fenvinline.c
+++ b/math/test-fenvinline.c
@@ -200,11 +200,11 @@ test_fesetround (void)
 #endif
 }
 
+#if FE_ALL_EXCEPT
 /* Tests for feenableexcept/fedisableexcept.  */
 static void
 feenable_test (const char *flag_name, fexcept_t fe_exc)
 {
-#if FE_ALL_EXCEPT
   int fe_exci = fe_exc;
   double fe_excd = fe_exc;
   int excepts;
@@ -287,8 +287,8 @@ feenable_test (const char *flag_name, fexcept_t fe_exc)
               flag_name, (unsigned int)fe_exc, excepts);
       ++count_errors;
     }
-#endif
 }
+#endif
 
 static void
 test_feenabledisable (void)