From 8a131190070152ffb0a27ab5dd14d98052467aa9 Mon Sep 17 00:00:00 2001 From: Sam McCall Date: Tue, 20 Sep 2022 22:17:02 +0200 Subject: [PATCH] [clangd] Remove second tracer (which breaks threading contract) --- clang-tools-extra/clangd/TUScheduler.cpp | 3 --- 1 file changed, 3 deletions(-) diff --git a/clang-tools-extra/clangd/TUScheduler.cpp b/clang-tools-extra/clangd/TUScheduler.cpp index 1e64fac..b85b8e5 100644 --- a/clang-tools-extra/clangd/TUScheduler.cpp +++ b/clang-tools-extra/clangd/TUScheduler.cpp @@ -395,11 +395,9 @@ public: // If there is no throttler, this dummy request is always satisfied. if (!Throttler) return; - Tracer.emplace("PreambleThrottled"); ID = Throttler->acquire(Filename, [&] { Satisfied.store(true, std::memory_order_release); CV.notify_all(); - Tracer.reset(); }); } @@ -414,7 +412,6 @@ public: } private: - llvm::Optional Tracer; PreambleThrottler::RequestID ID; PreambleThrottler *Throttler; std::atomic Satisfied = {false}; -- 2.7.4