[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