mib added inline comments.
================ Comment at: lldb/source/Plugins/DynamicLoader/Darwin-Kernel/DynamicLoaderDarwinKernel.cpp:806 + target.GetArchitecture(), + ConstString()); } ---------------- Can this be `{}` ? ================ Comment at: lldb/source/Target/Process.cpp:2389 } - ModuleSP module_sp(new Module(file_spec, ArchSpec())); + ModuleSP module_sp(new Module(file_spec, ArchSpec(), ConstString())); if (module_sp) { ---------------- Should the module constructor have this as a default value for the `ConstString` parameter ? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D158043/new/ https://reviews.llvm.org/D158043 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits