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

Lang Hames via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 2 22:15:02 PST 2024


================
@@ -893,14 +893,14 @@ Error MachOPlatform::MachOPlatformPlugin::
 
   for (auto *Sym : G.defined_symbols()) {
     for (auto &RTSym : RuntimeSymbols) {
-      if (Sym->hasName() && Sym->getName() == RTSym.first) {
+      if (Sym->hasName() && *Sym->getName() == RTSym.first) {
----------------
lhames wrote:

Should be `Sym->getName() == RTSym.first` (assuming we can `SymbolStringPtr`ify RTSym).

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


More information about the llvm-commits mailing list