[Lldb-commits] [PATCH] D41584: Check existence of each required component during construction of LLVMCDisassembler.

[Pavel Labath via Phabricator via lldb-commits]

labath added a comment.

In https://reviews.llvm.org/D41584#970945, @tatyana-krasnukha wrote:

> Thank you, Pavel.
>  Would you mind if I move LLVMCDisassembler declaration in .cpp also? It looks like perfect candidate for pimpl.


Yes, that sounds like a good idea.

> And... doesn't DisassemblerLLVMC::LLVMCDisassembler confuse anyone but me?)

Well... I haven't looked at this class in the past, but yes... it looks confusing. I wouldn't mind a name change.

Jason, any thoughts on this?


https://reviews.llvm.org/D41584