teemperor added inline comments.
================
Comment at: source/Plugins/Disassembler/llvm/DisassemblerLLVMC.cpp:176
bool got_op = false;
- std::shared_ptr<DisassemblerLLVMC> disasm_sp(GetDisassembler());
- if (disasm_sp) {
- const ArchSpec &arch = disasm_sp->GetArchitecture();
+ DisassemblerScope disasm(*this);
+ if (disasm) {
----------------
vsk wrote:
> Were all callers into Decode locking the disasm instance prior to this patch?
> If not, this is a sweet fix.
Not from what I can see. Otherwise I would have seen deadlocks when adding this
lock here.
Repository:
rL LLVM
https://reviews.llvm.org/D51319
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
