Remove Target::FindLoadAddrForNameInSymbolsAndPersistentVariables

This reverts commit 2542856. Remove
`Target::FindLoadAddrForNameInSymbolsAndPersistentVariables` as this is
going unused in the codebase. As such the implementation being used in
`FindSymbolForSwiftObject` is reverted to its original version.

Author: chelcassanova

lldb/include/lldb/Target/Target.h

@@ -1355,20 +1355,6 @@ class Target : public std::enable_shared_from_this<Target>,
       const EvaluateExpressionOptions &options = EvaluateExpressionOptions(),
       std::string *fixed_expression = nullptr, ValueObject *ctx_obj = nullptr);
 
-  lldb::addr_t FindLoadAddrForNameInSymbols(
-      ConstString name_const_str, lldb::SymbolType symbol_type);
-
-  lldb::addr_t FindLoadAddrForNameInPersistentVariables(
-    ConstString name_const_str, lldb::SymbolType symbol_type);
-
-  // Look up a symbol by name and type in both the target's symbols and the
-  // persistent symbols from the
-  // expression parser.  The symbol_type is ignored in that case, for now we
-  // don't have symbol types for the
-  // persistent variables.
-  lldb::addr_t FindLoadAddrForNameInSymbolsAndPersistentVariables(
-      ConstString name_const_str, lldb::SymbolType symbol_type);
-
   lldb::ExpressionVariableSP GetPersistentVariable(ConstString name);
 
   lldb::addr_t GetPersistentSymbol(ConstString name);

lldb/source/Plugins/LanguageRuntime/Swift/SwiftLanguageRuntime.cpp

@@ -180,9 +180,19 @@ FindSymbolForSwiftObject(Process &process, RuntimeKind runtime_kind,
     return {};
   }
 
+  SymbolContextList sc_list;
+  image->FindSymbolsWithNameAndType(ConstString(object), sym_type, sc_list);
+  if (sc_list.GetSize() != 1)
+    return {};
+
+  SymbolContext SwiftObject_Class;
+  if (!sc_list.GetContextAtIndex(0, SwiftObject_Class))
+    return {};
+  if (!SwiftObject_Class.symbol)
+    return {};
   lldb::addr_t addr =
-      target.FindLoadAddrForNameInSymbols(ConstString(object), sym_type);
-  if (addr != LLDB_INVALID_ADDRESS)
+      SwiftObject_Class.symbol->GetAddress().GetLoadAddress(&target);
+  if (addr && addr != LLDB_INVALID_ADDRESS)
     return addr;
 
   return {};

lldb/source/Target/Target.cpp

@@ -3693,70 +3693,6 @@ bool Target::SetSectionUnloaded(const lldb::SectionSP &section_sp,
 
 void Target::ClearAllLoadedSections() { m_section_load_history.Clear(); }
 
-lldb::addr_t Target::FindLoadAddrForNameInSymbols(ConstString name_const_str,
-                                                  SymbolType symbol_type) {
-  lldb::addr_t symbol_addr = LLDB_INVALID_ADDRESS;
-  SymbolContextList sc_list;
-
-  GetImages().FindSymbolsWithNameAndType(name_const_str, symbol_type, sc_list);
-  if (!sc_list.IsEmpty()) {
-    SymbolContext desired_symbol;
-
-    if (sc_list.GetSize() == 1 &&
-        sc_list.GetContextAtIndex(0, desired_symbol)) {
-      if (desired_symbol.symbol) {
-        symbol_addr = desired_symbol.symbol->GetAddress().GetLoadAddress(this);
-      }
-    } else if (sc_list.GetSize() > 1) {
-      for (size_t i = 0; i < sc_list.GetSize(); i++) {
-        if (sc_list.GetContextAtIndex(i, desired_symbol)) {
-          if (desired_symbol.symbol) {
-            symbol_addr =
-                desired_symbol.symbol->GetAddress().GetLoadAddress(this);
-            if (symbol_addr != LLDB_INVALID_ADDRESS)
-              break;
-          }
-        }
-      }
-    }
-  }
-  return symbol_addr;
-}
-
-lldb::addr_t
-Target::FindLoadAddrForNameInPersistentVariables(ConstString name_const_str,
-                                                 lldb::SymbolType symbol_type) {
-  lldb::addr_t symbol_addr = LLDB_INVALID_ADDRESS;
-  ConstString mangled_const_str;
-
-  if (name_const_str.GetMangledCounterpart(mangled_const_str)) {
-    symbol_addr = GetPersistentSymbol(mangled_const_str);
-  }
-
-  // Check in persistent variables using the name that was passed in
-  if (symbol_addr == LLDB_INVALID_ADDRESS) {
-    symbol_addr = GetPersistentSymbol(name_const_str);
-  }
-
-  return symbol_addr;
-}
-
-lldb::addr_t Target::FindLoadAddrForNameInSymbolsAndPersistentVariables(
-    ConstString name_const_str, SymbolType symbol_type) {
-  lldb::addr_t symbol_addr = LLDB_INVALID_ADDRESS;
-
-  // Try to find the name in the symbol list
-  symbol_addr =
-      Target::FindLoadAddrForNameInSymbols(name_const_str, symbol_type);
-
-  // If that doesn't work, look for it in persistent variables
-  if (symbol_addr == LLDB_INVALID_ADDRESS)
-    Target::FindLoadAddrForNameInPersistentVariables(name_const_str,
-                                                     symbol_type);
-
-  return symbol_addr;
-}
-
 void Target::SaveScriptedLaunchInfo(lldb_private::ProcessInfo &process_info) {
   if (process_info.IsScriptedProcess()) {
     // Only copy scripted process launch options.