================ @@ -62,15 +66,25 @@ bool canUseDebuginfod() { } SmallVector<StringRef> getDefaultDebuginfodUrls() { - const char *DebuginfodUrlsEnv = std::getenv("DEBUGINFOD_URLS"); - if (DebuginfodUrlsEnv == nullptr) - return SmallVector<StringRef>(); - - SmallVector<StringRef> DebuginfodUrls; - StringRef(DebuginfodUrlsEnv).split(DebuginfodUrls, " "); + if (!DebuginfodUrlsSet) { + // Only read from the environment variable if the user hasn't already + // set the value + const char *DebuginfodUrlsEnv = std::getenv("DEBUGINFOD_URLS"); + if (DebuginfodUrlsEnv != nullptr) { + StringRef(DebuginfodUrlsEnv).split(DebuginfodUrls, " ", -1, false); + } + DebuginfodUrlsSet = true; + } return DebuginfodUrls; } +// Override the default debuginfod URL list. +void setDefaultDebuginfodUrls(SmallVector<StringRef> URLs) { + DebuginfodUrls.clear(); ---------------- kevinfrei wrote:
Switched the setting to an array of strings setting, which addresses the usage issues. https://github.com/llvm/llvm-project/pull/70996 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits