[Lldb-commits] [lldb] [lldb] Enable locate module callback for main executable (PR #160199)

[Greg Clayton via lldb-commits]

================
@@ -767,9 +763,18 @@ Platform::ResolveExecutable(const ModuleSpec &module_spec,
   Status error;
   for (const ArchSpec &arch : GetSupportedArchitectures(process_host_arch)) {
     resolved_module_spec.GetArchitecture() = arch;
-    error =
-        ModuleList::GetSharedModule(resolved_module_spec, exe_module_sp,
-                                    module_search_paths_ptr, nullptr, nullptr);
+
+    // Call locate module callback first, then fallback to standard path.
+    FileSpec symbol_file_spec;
+    CallLocateModuleCallbackIfSet(resolved_module_spec, exe_module_sp,
+                                  symbol_file_spec, nullptr);
----------------
clayborg wrote:

Do we really want to always call the locate module callback first instead of just letting `ModuleList::GetSharedModule` call it after it searches the current global module list? That seems wrong to me

https://github.com/llvm/llvm-project/pull/160199