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

[Jared Wyles via llvm-commits]

================
@@ -1415,10 +1454,10 @@ class LinkGraph {
   void makeAbsolute(Symbol &Sym, orc::ExecutorAddr Address) {
     assert(!Sym.isAbsolute() && "Symbol is already absolute");
     if (Sym.isExternal()) {
-      assert(ExternalSymbols.contains(Sym.getName()) &&
+      assert(ExternalSymbols.contains(*Sym.getName()) &&
----------------
jaredwy wrote:

Yup this ones on my list of come back after this initial patch lands. Ill drop in a fix me 

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