jingham added inline comments.
================ Comment at: lldb/source/Plugins/LanguageRuntime/ObjC/AppleObjCRuntime/AppleObjCRuntimeV2.cpp:2359 +void AppleObjCRuntimeV2::WarnIfNoExpandedSharedCache() { +#ifdef __APPLE__ + if (!m_objc_module_sp) ---------------- JDevlieghere wrote: > jingham wrote: > > Why is this code #ifdef APPLE? > > > > If you were debugging from a Linux machine to a macOS system, wouldn't you > > also want to at least see the warning so you would know why their session > > was slow? > My reasoning was that the error wouldn't be actionable so I'd rather not show > it all. I don't feel strongly about it though. It would save people wasting time to try to figure out why this seemed slow, which is worth something on its own. And the Linux user could expand the cache on the Mac, copy it over and lldb should be able to use it - with "target modules add" by hand if nothing more automatic works. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D117623/new/ https://reviews.llvm.org/D117623 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits