[Lldb-commits] [lldb] 4bd4650 - Use llvm::any_of and llvm::none_of (NFC)

2021-10-24 Thread Kazu Hirata via lldb-commits
Author: Kazu Hirata Date: 2021-10-24T17:35:33-07:00 New Revision: 4bd46501c394ce221d34b60155f48ebbf6b6897d URL: https://github.com/llvm/llvm-project/commit/4bd46501c394ce221d34b60155f48ebbf6b6897d DIFF: https://github.com/llvm/llvm-project/commit/4bd46501c394ce221d34b60155f48ebbf6b6897d.diff

[Lldb-commits] [PATCH] D112212: [lldb/test] Print build commands in trace mode

2021-10-24 Thread David Blaikie via Phabricator via lldb-commits
dblaikie added a comment. In D112212#3082352 , @JDevlieghere wrote: > In D112212#3081935 , @dblaikie > wrote: > >> In D112212#3081828 , @JDevlieghere >> wrote: >> >>>

[Lldb-commits] [lldb] 7cc8fa2 - Use llvm::is_contained (NFC)

2021-10-24 Thread Kazu Hirata via lldb-commits
Author: Kazu Hirata Date: 2021-10-24T09:32:57-07:00 New Revision: 7cc8fa2dd2d8167d427cadb4fc6f4254b53a8359 URL: https://github.com/llvm/llvm-project/commit/7cc8fa2dd2d8167d427cadb4fc6f4254b53a8359 DIFF: https://github.com/llvm/llvm-project/commit/7cc8fa2dd2d8167d427cadb4fc6f4254b53a8359.diff

[Lldb-commits] [PATCH] D112379: [lldb][NFC] Modernize for-loops in ModuleList

2021-10-24 Thread Med Ismail Bennani via Phabricator via lldb-commits
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-commits] [PATCH] D112379: [lldb][NFC] Modernize for-loops in ModuleList

2021-10-24 Thread Raphael Isemann via Phabricator via lldb-commits
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 _module, bool notify) { + if (new_module) { std::lock_guard guard(m_modules_mutex);

[Lldb-commits] [PATCH] D112379: [lldb][NFC] Modernize for-loops in ModuleList

2021-10-24 Thread Raphael Isemann via Phabricator via lldb-commits
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