clayborg accepted this revision.
clayborg added a comment.
LGTM with a few nits that can be fixed if needed
================
Comment at: lldb/source/Target/Process.cpp:5762-5764
if (sys_runtime) {
sys_runtime->ModulesDidLoad(module_list);
}
----------------
remove braces?
================
Comment at: lldb/source/Target/Process.cpp:5772-5774
+ for (auto &runtime : m_instrumentation_runtimes) {
+ runtime.second->ModulesDidLoad(module_list);
}
----------------
remove braces now that this is just a single line
Repository:
rLLDB LLDB
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D84475/new/
https://reviews.llvm.org/D84475
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
