Commit 1026cd68 authored by Leszek Swirski's avatar Leszek Swirski Committed by V8 LUCI CQ

[test] Make RCS test time atomic to fix TSAN

Change-Id: Ibc256e202a75f7dbcba27dad2b591cf45738b005
Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3571900
Commit-Queue: Leszek Swirski <leszeks@chromium.org>
Auto-Submit: Leszek Swirski <leszeks@chromium.org>
Reviewed-by: 's avatarCamillo Bruni <cbruni@chromium.org>
Commit-Queue: Camillo Bruni <cbruni@chromium.org>
Cr-Commit-Position: refs/heads/main@{#79809}
parent 3bb9ab56
...@@ -4,6 +4,8 @@ ...@@ -4,6 +4,8 @@
#include "src/logging/runtime-call-stats.h" #include "src/logging/runtime-call-stats.h"
#include <atomic>
#include "include/v8-template.h" #include "include/v8-template.h"
#include "src/api/api-inl.h" #include "src/api/api-inl.h"
#include "src/base/atomic-utils.h" #include "src/base/atomic-utils.h"
...@@ -21,7 +23,8 @@ namespace internal { ...@@ -21,7 +23,8 @@ namespace internal {
namespace { namespace {
static base::TimeTicks runtime_call_stats_test_time_ = base::TimeTicks(); static std::atomic<base::TimeTicks> runtime_call_stats_test_time_ =
base::TimeTicks();
// Time source used for the RuntimeCallTimer during tests. We cannot rely on // Time source used for the RuntimeCallTimer during tests. We cannot rely on
// the native timer since it's too unpredictable on the build bots. // the native timer since it's too unpredictable on the build bots.
static base::TimeTicks RuntimeCallStatsTestNow() { static base::TimeTicks RuntimeCallStatsTestNow() {
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment