[llvm] Move a lot of symbol code to use the symbol string pool (PR #115796)

[Jared Wyles via llvm-commits]

================
@@ -150,7 +151,7 @@ class RuntimeDyldChecker {
 
   using IsSymbolValidFunction = std::function<bool(StringRef Symbol)>;
   using GetSymbolInfoFunction =
-      std::function<Expected<MemoryRegionInfo>(StringRef SymbolName)>;
+      std::function<Expected<MemoryRegionInfo>(StringRef Symbol)>;
   using GetSectionInfoFunction = std::function<Expected<MemoryRegionInfo>(
----------------
jaredwy wrote:

It makes it consistent with the rest of our usages in this file. Ill swing back through and change it in a future patch


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