[PATCH] D143428: [clang][deps] Fix race condition
Jan Svoboda via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Feb 7 09:27:50 PST 2023
This revision was automatically updated to reflect the committed changes.
Closed by commit rG152f2914607f: [clang][deps] Fix race condition (authored by jansvoboda11).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D143428/new/
https://reviews.llvm.org/D143428
Files:
clang/tools/clang-scan-deps/ClangScanDeps.cpp
Index: clang/tools/clang-scan-deps/ClangScanDeps.cpp
===================================================================
--- clang/tools/clang-scan-deps/ClangScanDeps.cpp
+++ clang/tools/clang-scan-deps/ClangScanDeps.cpp
@@ -252,14 +252,17 @@
public:
void mergeDeps(StringRef Input, TranslationUnitDeps TUDeps,
size_t InputIndex) {
+ mergeDeps(std::move(TUDeps.ModuleGraph), InputIndex);
+
InputDeps ID;
ID.FileName = std::string(Input);
ID.ContextHash = std::move(TUDeps.ID.ContextHash);
ID.FileDeps = std::move(TUDeps.FileDeps);
ID.ModuleDeps = std::move(TUDeps.ClangModuleDeps);
- mergeDeps(std::move(TUDeps.ModuleGraph), InputIndex);
ID.DriverCommandLine = std::move(TUDeps.DriverCommandLine);
ID.Commands = std::move(TUDeps.Commands);
+
+ std::unique_lock<std::mutex> ul(Lock);
Inputs.push_back(std::move(ID));
}
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D143428.495576.patch
Type: text/x-patch
Size: 894 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20230207/2bef543b/attachment-0001.bin>
More information about the cfe-commits
mailing list