[Lldb-commits] [PATCH] D53094: [pecoff] Implement ObjectFilePECOFF::GetDependedModules()
Zachary Turner via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Wed Oct 10 11:27:54 PDT 2018
zturner added a comment.
If you plan to invest in more substantial changes in `ObjectFilePECOFF`, it might worth considering a complete re-write in terms of `llvm::object::coff`. It has pretty comprehensive support for the PE/COFF spec, so it's just a matter of implementing `ObjectFilePECOFF` on top of it.
================
Comment at: source/Plugins/ObjectFile/PECOFF/ObjectFilePECOFF.cpp:810
+
+ std::lock_guard<std::recursive_mutex> guard(module_sp->GetMutex());
+
----------------
Does this actually need to be a `recursive_mutex`?
Repository:
rLLDB LLDB
https://reviews.llvm.org/D53094
More information about the lldb-commits
mailing list