JDevlieghere accepted this revision. JDevlieghere added a comment. This revision is now accepted and ready to land.
LGTM modulo the redundant comment ================ Comment at: lldb/unittests/Target/RemoteAwarePlatformTest.cpp:40 const ModuleSpec &module_spec, lldb::ModuleSP &exe_module_sp, - const FileSpecList *module_search_paths_ptr) /*override*/ { + const FileSpecList *module_search_paths_ptr) override /*override*/ { auto pair = ResolveRemoteExecutable(module_spec, module_search_paths_ptr); ---------------- CHANGES SINCE LAST ACTION https://reviews.llvm.org/D123340/new/ https://reviews.llvm.org/D123340 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits