From 61a16874e56048814b68ed49fdea58362551d9be Mon Sep 17 00:00:00 2001 From: zhijian Date: Tue, 26 Sep 2023 15:38:11 -0400 Subject: [PATCH 1/2] [NFC] refactor demangle of llvm-nm --- llvm/tools/llvm-nm/llvm-nm.cpp | 35 ++-------------------------------- 1 file changed, 2 insertions(+), 33 deletions(-) diff --git a/llvm/tools/llvm-nm/llvm-nm.cpp b/llvm/tools/llvm-nm/llvm-nm.cpp index 051fa3e5bfa5a..6812a90c4cfae 100644 --- a/llvm/tools/llvm-nm/llvm-nm.cpp +++ b/llvm/tools/llvm-nm/llvm-nm.cpp @@ -629,30 +629,6 @@ static void darwinPrintStab(MachOObjectFile *MachO, const NMSymbol &S) { outs() << format(" %02x", NType); } -static std::optional demangle(StringRef Name) { - std::string Demangled; - if (nonMicrosoftDemangle(Name, Demangled)) - return Demangled; - return std::nullopt; -} - -static std::optional demangleXCOFF(StringRef Name) { - if (Name.empty() || Name[0] != '.') - return demangle(Name); - - Name = Name.drop_front(); - std::optional DemangledName = demangle(Name); - if (DemangledName) - return "." + *DemangledName; - return std::nullopt; -} - -static std::optional demangleMachO(StringRef Name) { - if (!Name.empty() && Name[0] == '_') - Name = Name.drop_front(); - return demangle(Name); -} - static bool symbolIsDefined(const NMSymbol &Sym) { return Sym.TypeChar != 'U' && Sym.TypeChar != 'w' && Sym.TypeChar != 'v'; } @@ -822,15 +798,8 @@ static void printSymbolList(SymbolicFile &Obj, std::string Name = S.Name; MachOObjectFile *MachO = dyn_cast(&Obj); - if (Demangle) { - function_ref(StringRef)> Fn = ::demangle; - if (Obj.isXCOFF()) - Fn = demangleXCOFF; - if (Obj.isMachO()) - Fn = demangleMachO; - if (std::optional Opt = Fn(S.Name)) - Name = *Opt; - } + if (Demangle) + Name = llvm::demangle(Name); if (PrintFileName) writeFileName(outs(), ArchiveName, ArchitectureName); From 2e93a6f108c18a1e9732d38f08d7411bb040cf92 Mon Sep 17 00:00:00 2001 From: zhijian Date: Wed, 27 Sep 2023 10:58:28 -0400 Subject: [PATCH 2/2] address comment --- llvm/tools/llvm-nm/llvm-nm.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/llvm/tools/llvm-nm/llvm-nm.cpp b/llvm/tools/llvm-nm/llvm-nm.cpp index 6812a90c4cfae..e32aa8ab8f5bf 100644 --- a/llvm/tools/llvm-nm/llvm-nm.cpp +++ b/llvm/tools/llvm-nm/llvm-nm.cpp @@ -799,7 +799,7 @@ static void printSymbolList(SymbolicFile &Obj, std::string Name = S.Name; MachOObjectFile *MachO = dyn_cast(&Obj); if (Demangle) - Name = llvm::demangle(Name); + Name = demangle(Name); if (PrintFileName) writeFileName(outs(), ArchiveName, ArchitectureName);