This revision was automatically updated to reflect the committed changes.
Closed by commit rG4cf9d1e4492f: [lldb][NFC] Modernize for-loops in ModuleList
(authored by teemperor).
Herald added a subscriber: lldb-commits.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://rev
mib accepted this revision as: mib.
mib added a comment.
This revision is now accepted and ready to land.
LGTM
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D112379/new/
https://reviews.llvm.org/D112379
___
lldb-commits mailing list
lldb-commi
teemperor added a comment.
Straightforward but looking for a second set of eyes
Comment at: lldb/source/Core/ModuleList.cpp:204
+bool ModuleList::AppendIfNeeded(const ModuleSP &new_module, bool notify) {
+ if (new_module) {
std::lock_guard guard(m_modules_mutex);
teemperor created this revision.
teemperor added a reviewer: LLDB.
teemperor added a project: LLDB.
Herald added a subscriber: JDevlieghere.
teemperor requested review of this revision.
https://reviews.llvm.org/D112379
Files:
lldb/include/lldb/Core/ModuleList.h
lldb/source/Core/ModuleList.cpp