[llvm] [DTLTO] [LLVM] Initial DTLTO cache implementation (PR #156433)

Katya Romanova via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 11 14:52:13 PDT 2025


================
@@ -2244,14 +2245,17 @@ class OutOfProcessThinBackend : public CGThinBackend {
 
   SmallVector<StringRef, 0> CodegenOptions;
   DenseSet<StringRef> CommonInputs;
-
+  std::atomic<uint64_t> CachedJobs{0};
----------------
romanova-ekaterina wrote:

Are you suggesting to change it to 
 `std::atomic<size_t> CachedJobs{0};`
 or to 
 `size_t CachedJobs{0};`
 
 This variable is changed in the thread pool.  We cannot safely get rid of atomic because a data race will get introduced.
 

https://github.com/llvm/llvm-project/pull/156433


More information about the llvm-commits mailing list