[Lldb-commits] [lldb] [llvm] DEBUGINFOD based DWP acquisition for LLDB (PR #70996)

[Greg Clayton via lldb-commits]

================
@@ -47,6 +47,10 @@ namespace llvm {
 
 using llvm::object::BuildIDRef;
 
+SmallVector<StringRef> DebuginfodUrls;
----------------
clayborg wrote:

Maybe make this `std::optional< SmallVector<StringRef> >` and get rid of `DebuginfodUrlsSet` below? We might want a std::mutex here to protect multi-threaded access to the `DebuginfodUrls` variable.

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