[lld] r317014 - [ELF] Remove unnecessary function call. NFC.

James Henderson via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 31 10:28:44 PDT 2017


Author: jhenderson
Date: Tue Oct 31 10:28:44 2017
New Revision: 317014

URL: http://llvm.org/viewvc/llvm-project?rev=317014&view=rev
Log:
[ELF] Remove unnecessary function call. NFC.

StringTableSection does not implement postThunkContents, so calling it on these 
sections is pointless (it calls an empty virtual function), and we can remove it.

Reviewers: ruiu

Differential Revision: https://reviews.llvm.org/D39460

Modified:
    lld/trunk/ELF/Writer.cpp

Modified: lld/trunk/ELF/Writer.cpp
URL: http://llvm.org/viewvc/llvm-project/lld/trunk/ELF/Writer.cpp?rev=317014&r1=317013&r2=317014&view=diff
==============================================================================
--- lld/trunk/ELF/Writer.cpp (original)
+++ lld/trunk/ELF/Writer.cpp Tue Oct 31 10:28:44 2017
@@ -1363,7 +1363,7 @@ template <class ELFT> void Writer<ELFT>:
     Sec->finalize<ELFT>();
 
   // createThunks may have added local symbols to the static symbol table
-  applySynthetic({InX::SymTab, InX::ShStrTab, InX::StrTab},
+  applySynthetic({InX::SymTab},
                  [](SyntheticSection *SS) { SS->postThunkContents(); });
 }
 




More information about the llvm-commits mailing list