[lld] r344089 - Merge two overloaded functions into one function. NFC.

Rui Ueyama via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 9 15:44:53 PDT 2018


Author: ruiu
Date: Tue Oct  9 15:44:53 2018
New Revision: 344089

URL: http://llvm.org/viewvc/llvm-project?rev=344089&view=rev
Log:
Merge two overloaded functions into one function. NFC.

Modified:
    lld/trunk/ELF/SymbolTable.cpp

Modified: lld/trunk/ELF/SymbolTable.cpp
URL: http://llvm.org/viewvc/llvm-project/lld/trunk/ELF/SymbolTable.cpp?rev=344089&r1=344088&r2=344089&view=diff
==============================================================================
--- lld/trunk/ELF/SymbolTable.cpp (original)
+++ lld/trunk/ELF/SymbolTable.cpp Tue Oct  9 15:44:53 2018
@@ -442,12 +442,6 @@ Symbol *SymbolTable::addCommon(StringRef
   return S;
 }
 
-static void reportDuplicate(Symbol *Sym, InputFile *NewFile) {
-  if (!Config->AllowMultipleDefinition)
-    error("duplicate symbol: " + toString(*Sym) + "\n>>> defined in " +
-          toString(Sym->File) + "\n>>> defined in " + toString(NewFile));
-}
-
 static void reportDuplicate(Symbol *Sym, InputFile *NewFile,
                             InputSectionBase *ErrSec, uint64_t ErrOffset) {
   if (Config->AllowMultipleDefinition)
@@ -455,7 +449,8 @@ static void reportDuplicate(Symbol *Sym,
 
   Defined *D = cast<Defined>(Sym);
   if (!D->Section || !ErrSec) {
-    reportDuplicate(Sym, NewFile);
+    error("duplicate symbol: " + toString(*Sym) + "\n>>> defined in " +
+          toString(Sym->File) + "\n>>> defined in " + toString(NewFile));
     return;
   }
 
@@ -544,7 +539,7 @@ Symbol *SymbolTable::addBitcode(StringRe
   if (Cmp > 0)
     replaceSymbol<Defined>(S, &F, Name, Binding, StOther, Type, 0, 0, nullptr);
   else if (Cmp == 0)
-    reportDuplicate(S, &F);
+    reportDuplicate(S, &F, nullptr, 0);
   return S;
 }
 




More information about the llvm-commits mailing list