Delete dead code.
authorRafael Espindola <rafael.espindola@gmail.com>
Thu, 8 Dec 2016 03:17:05 +0000 (03:17 +0000)
committerRafael Espindola <rafael.espindola@gmail.com>
Thu, 8 Dec 2016 03:17:05 +0000 (03:17 +0000)
Thanks to George Rimar for pointing it out.

llvm-svn: 289020

lld/ELF/LinkerScript.cpp
lld/ELF/Writer.cpp

index f126ef0..8a0c39d 100644 (file)
@@ -764,19 +764,6 @@ void LinkerScript<ELFT>::assignAddresses(std::vector<PhdrEntry<ELFT>> &Phdrs) {
   MinVA = alignDown(MinVA - HeaderSize, Config->MaxPageSize);
   Out<ELFT>::ElfHeader->Addr = MinVA;
   Out<ELFT>::ProgramHeaders->Addr = Out<ELFT>::ElfHeader->Size + MinVA;
-
-  if (!FirstPTLoad->First) {
-    // Sometimes the very first PT_LOAD segment can be empty.
-    // This happens if (all conditions met):
-    //  - Linker script is used
-    //  - First section in ELF image is not RO
-    //  - Not enough space for program headers.
-    // The code below removes empty PT_LOAD segment and updates
-    // program headers size.
-    Phdrs.erase(FirstPTLoad);
-    Out<ELFT>::ProgramHeaders->Size =
-        sizeof(typename ELFT::Phdr) * Phdrs.size();
-  }
 }
 
 // Creates program headers as instructed by PHDRS linker script command.
index d7df3ac..ce1e533 100644 (file)
@@ -139,6 +139,8 @@ template <class ELFT> void Writer<ELFT>::removeEmptyPTLoad() {
   auto I = std::remove_if(Phdrs.begin(), Phdrs.end(), [&](const Phdr &P) {
     if (P.H.p_type != PT_LOAD)
       return false;
+    if (!P.First)
+      return true;
     uintX_t Size = P.Last->Addr + P.Last->Size - P.First->Addr;
     return Size == 0;
   });