[Lldb-commits] [lldb] [lldb] Expand background symbol lookup (PR #80890)

[Jason Molenda via lldb-commits]

================
@@ -5,10 +5,11 @@ let Definition = "modulelist" in {
     Global,
     DefaultTrue,
     Desc<"Control the use of external tools and repositories to locate symbol files. Directories listed in target.debug-file-search-paths and directory of the executable are always checked first for separate debug info files. Then depending on this setting: On macOS, Spotlight would be also used to locate a matching .dSYM bundle based on the UUID of the executable. On NetBSD, directory /usr/libdata/debug would be also searched. On platforms other than NetBSD directory /usr/lib/debug would be also searched. If all other methods fail there may be symbol-locator plugins that, if configured properly, will also attempt to acquire symbols. The debuginfod plugin defaults to the DEGUFINFOD_URLS environment variable which is configurable through the 'plugin.symbol-locator.debuginfod.server_urls' setting.">;
-  def EnableBackgroundLookup: Property<"enable-background-lookup", "Boolean">,
+  def LazySymbolLookup: Property<"lazy-lookup", "Enum">,
----------------
jasonmolenda wrote:

(just to be clear, my fear of `symbols.download-on-demand` in addition to the existing `symbols.load-on-demand` is that no one except us will ever understand what the difference is.  I can sit here and say "it's very clear if you think about it!" but I think it's not worth it to be this subtle, IMO.)

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