diff options
author | 2014-07-15 19:04:14 -0400 | |
---|---|---|
committer | 2014-07-15 19:04:14 -0400 | |
commit | 7edaeb515c7b33ff965f301322d0477c6d11c728 (patch) | |
tree | 0948cb9e034d31f566bb084405f5ce5e64d449cb /tools | |
parent | f7e51ea79e771533ff44f3c09a0bb01af0298b38 (diff) |
nanobench on Windows: try compiler barriers around timer instead.
BUG=skia:
Review URL: https://codereview.chromium.org/391193003
Diffstat (limited to 'tools')
-rw-r--r-- | tools/timer/SysTimer_windows.cpp | 24 |
1 files changed, 16 insertions, 8 deletions
diff --git a/tools/timer/SysTimer_windows.cpp b/tools/timer/SysTimer_windows.cpp index 2f9d0a5d58..d24a6aae71 100644 --- a/tools/timer/SysTimer_windows.cpp +++ b/tools/timer/SysTimer_windows.cpp @@ -6,6 +6,8 @@ */ #include "SysTimer_windows.h" +#include <intrin.h> + static ULONGLONG win_cpu_time() { FILETIME createTime; FILETIME exitTime; @@ -23,11 +25,6 @@ static ULONGLONG win_cpu_time() { return start_cpu_sys.QuadPart + start_cpu_usr.QuadPart; } -void SysTimer::startWall() { - if (0 == ::QueryPerformanceCounter(&fStartWall)) { - fStartWall.QuadPart = 0; - } -} void SysTimer::startCpu() { fStartCpu = win_cpu_time(); } @@ -36,11 +33,22 @@ double SysTimer::endCpu() { ULONGLONG end_cpu = win_cpu_time(); return static_cast<double>(end_cpu - fStartCpu) / 10000.0L; } + +static void wall_timestamp(LARGE_INTEGER* now) { + _ReadWriteBarrier(); + if (0 == ::QueryPerformanceCounter(now)) { + now->QuadPart = 0; + } + _ReadWriteBarrier(); +} + +void SysTimer::startWall() { + wall_timestamp(&fStartWall); +} + double SysTimer::endWall() { LARGE_INTEGER end_wall; - if (0 == ::QueryPerformanceCounter(&end_wall)) { - end_wall.QuadPart = 0; - } + wall_timestamp(&end_wall); LARGE_INTEGER ticks_elapsed; ticks_elapsed.QuadPart = end_wall.QuadPart - fStartWall.QuadPart; |