RepoMirrors/gperftools
1
0
Fork 0
mirror of https://github.com/gperftools/gperftools synced 2025-03-11 07:17:38 +00:00
Code Issues Packages Projects Releases Wiki Activity

slow down profiler_unittest some more

Browse Source 
On modern machines xoring volatile int is fast enough to occasionally
make specific test case's profiling results not stable enough. So we
now force atomics into the picture which is slightly slower (could be
lots on some platforms and/or optimizer flags).
This commit is contained in:
Aliaksey Kandratsenka 2023-06-21 10:37:58 -04:00
parent c37f6c4c7c
commit a80ce7b0e1
1 changed files with 10 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
src/tests/profiler_unittest.cc
View File

@ -42,12 +42,15 @@
#include <unistd.h> // for fork()
#endif
#include <sys/wait.h> // for wait()
#include <atomic>
#include "gperftools/profiler.h"
#include "base/simple_mutex.h"
#include "tests/testutil.h"
static volatile int result = 0;
static int g_iters = 0; // argv[1]
static std::atomic<int> result;
static int g_iters; // argv[1]
Mutex mutex(Mutex::LINKER_INITIALIZED);
@ -62,7 +65,9 @@ static void test_other_thread() {
for (i = 0; i < g_iters; ++i ) {
result ^= i;
}
snprintf(b, sizeof(b), "other: %d", result); // get some libc action
snprintf(b, sizeof(b), "other: %d", result.load()); // get some libc action
(void)noopt(b); // 'consume' b. Ensure that smart compiler doesn't
// remove snprintf call
}
#endif
}
@ -75,7 +80,8 @@ static void test_main_thread() {
for (i = 0; i < g_iters; ++i ) {
result ^= i;
}
snprintf(b, sizeof(b), "same: %d", result); // get some libc action
snprintf(b, sizeof(b), "same: %d", result.load()); // get some libc action
(void)noopt(b); // 'consume' b
}
}