diff mbox

[COMMITTED] math/test-fenvinline: avoid compiler warning

Message ID 1436559214-21077-1-git-send-email-cmetcalf@ezchip.com
State Committed
Headers

Commit Message

Chris Metcalf July 10, 2015, 8:13 p.m. UTC 
  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.
---
 ChangeLog              |    5 +++++
 math/test-fenvinline.c |    4 ++--
 2 files changed, 7 insertions(+), 2 deletions(-)
diff mbox

Patch

diff --git a/ChangeLog b/ChangeLog
index 0c7fc2c..8188ee3 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 b02cefd..d196268 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)