[lld] r368550 - [ELF] Remove unnecessary assignment to `isPreemptible` in replaceWithDefined()

Fangrui Song via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 12 03:01:17 PDT 2019


Author: maskray
Date: Mon Aug 12 03:01:17 2019
New Revision: 368550

URL: http://llvm.org/viewvc/llvm-project?rev=368550&view=rev
Log:
[ELF] Remove unnecessary assignment to `isPreemptible` in replaceWithDefined()

After r368535, it is no longer used in the handling of VER_NDX_LOCAL.
Drop it.

Modified:
    lld/trunk/ELF/Relocations.cpp

Modified: lld/trunk/ELF/Relocations.cpp
URL: http://llvm.org/viewvc/llvm-project/lld/trunk/ELF/Relocations.cpp?rev=368550&r1=368549&r2=368550&view=diff
==============================================================================
--- lld/trunk/ELF/Relocations.cpp (original)
+++ lld/trunk/ELF/Relocations.cpp Mon Aug 12 03:01:17 2019
@@ -510,7 +510,6 @@ static void replaceWithDefined(Symbol &s
   sym.gotIndex = old.gotIndex;
   sym.verdefIndex = old.verdefIndex;
   sym.ppc64BranchltIndex = old.ppc64BranchltIndex;
-  sym.isPreemptible = true;
   sym.exportDynamic = true;
   sym.isUsedInRegularObj = true;
 }




More information about the llvm-commits mailing list