This revision was automatically updated to reflect the committed changes. Closed by commit rCTE340156: [clangd] Add missing lock in the lookup. (authored by hokein, committed by ).
Changed prior to commit: https://reviews.llvm.org/D50960?vs=161442&id=161447#toc Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D50960 Files: clangd/index/MemIndex.cpp Index: clangd/index/MemIndex.cpp =================================================================== --- clangd/index/MemIndex.cpp +++ clangd/index/MemIndex.cpp @@ -64,6 +64,7 @@ void MemIndex::lookup(const LookupRequest &Req, llvm::function_ref<void(const Symbol &)> Callback) const { + std::lock_guard<std::mutex> Lock(Mutex); for (const auto &ID : Req.IDs) { auto I = Index.find(ID); if (I != Index.end())
Index: clangd/index/MemIndex.cpp =================================================================== --- clangd/index/MemIndex.cpp +++ clangd/index/MemIndex.cpp @@ -64,6 +64,7 @@ void MemIndex::lookup(const LookupRequest &Req, llvm::function_ref<void(const Symbol &)> Callback) const { + std::lock_guard<std::mutex> Lock(Mutex); for (const auto &ID : Req.IDs) { auto I = Index.find(ID); if (I != Index.end())
_______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits