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 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
