[Lldb-commits] [lldb] [lldb] Show module name in progress update for downloading symbols (PR #85342)

[Jonas Devlieghere via lldb-commits]

================
@@ -1066,11 +1066,21 @@ bool SymbolLocatorDebugSymbols::DownloadObjectAndSymbolFile(
   command << lookup_arg;
 
   // Log and report progress.
+  std::string lookup_desc;
+  if (uuid_ptr && file_spec_ptr)
+    lookup_desc =
+        llvm::formatv("{0} ({1})", file_spec_ptr->GetFilename().GetString(),
+                      uuid_ptr->GetAsString());
+  else if (uuid_ptr)
+    lookup_desc = uuid_ptr->GetAsString();
+  else if (file_spec_ptr)
+    lookup_desc = file_spec_ptr->GetFilename().GetString();
+
   Log *log = GetLog(LLDBLog::Host);
   LLDB_LOG(log, "Calling {0} with {1} to find dSYM: {2}", dsymForUUID_exe_path,
-           lookup_arg, command.GetString());
+           lookup_desc, command.GetString());
 
-  Progress progress("Downloading symbol file", lookup_arg);
+  Progress progress("Downloading symbol file", lookup_desc);
----------------
JDevlieghere wrote:

The 3 possible status messages are:

```
Downloading symbol file: libfoo.dylib (00000000-0000-0000-0000-000000000000)
Downloading symbol file: 00000000-0000-0000-0000-000000000000
Downloading symbol file: libfoo.dylib
```

For when we have both the file and UUID, only the UUID or only the file. 

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