This is the mail archive of the libc-alpha@sourceware.org mailing list for the glibc project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

[PATCH] Use volatile global counters in test-tgmath.c.


Hi,

If build with -Os on s390x, the test-tgmath fails with:
float functions not called often enough (-10000)

Within compile_testf(), the counter (count_float) is saved
before the complex functions are called.
Afterwards the saved counter differs to the current-counter.

But the tests with the complex functions do not increment count_float!
Instead count_float is saved to a register before calling totalorder
and totalordermag which both increment count_float.
The compiler is allowed to do that as totalorderf and totalordermagf
is declared with __attribute__ ((__const__)) in math/bits/mathcalls.h.

Thus this patch adjusts the global counters to be volatile.
Then count_float is saved after totalordermag.

Okay to commit?

Bye
Stefan

---
ChangeLog:

	* math/test-tgmath.c (count_double, count_float,
	count_ ldouble,	count_cdouble, count_cfloat,
	count_cldouble): Use volatile int.
commit 03fd4e7e0fd1d2f6dedee49bdbefde7b23a644e1
Author: Stefan Liebler <stli@linux.vnet.ibm.com>
Date:   Mon Apr 16 16:55:25 2018 +0200

    Use volatile global counters in test-tgmath.c.
    
    If build with -Os on s390x, the test-tgmath fails with:
    float functions not called often enough (-10000)
    
    Within compile_testf(), the counter (count_float) is saved
    before the complex functions are called.
    Afterwards the saved counter differs to the current-counter.
    
    But the tests with the complex functions do not increment count_float!
    Instead count_float is saved to a register before calling totalorder
    and totalordermag which both increment count_float.
    The compiler is allowed to do that as totalorderf and totalordermagf
    is declared with __attribute__ ((__const__)) in math/bits/mathcalls.h.
    
    Thus this patch adjusts the global counters to be volatile.
    Then count_float is saved after totalordermag.
    
    ChangeLog:
    
    	* math/test-tgmath.c (count_double, count_float,
    	count_ ldouble,	count_cdouble, count_cfloat,
    	count_cldouble): Use volatile int.

diff --git a/math/test-tgmath.c b/math/test-tgmath.c
index 6d8e7fd..2b0787c 100644
--- a/math/test-tgmath.c
+++ b/math/test-tgmath.c
@@ -45,12 +45,12 @@ complex float fz;
 complex double dz;
 complex long double lz;
 
-int count_double;
-int count_float;
-int count_ldouble;
-int count_cdouble;
-int count_cfloat;
-int count_cldouble;
+volatile int count_double;
+volatile int count_float;
+volatile int count_ldouble;
+volatile int count_cdouble;
+volatile int count_cfloat;
+volatile int count_cldouble;
 
 #define NCALLS     134
 #define NCALLS_INT 4

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]