[ELF] - Move calculation of _end to fixAbsoluteSymbols()

[grimar]

Summary

That is consistent with other symbols: _edata, _etext
and can help to avoid duplicate code.

[ruiu]

LGTM

ELF/Writer.cpp

if (needsPtLoad<ELFT>(Sec)) {
       ThreadBssOffset = TVA - VA + Sec->getSize();
     }
   }
 
   // Add space for section headers.
   SectionHeaderOff = alignTo(FileOff, sizeof(uintX_t));
   FileSize = SectionHeaderOff + getNumSections() * sizeof(Elf_Shdr);
 
-  // Update "_end" and "end" symbols so that they
-  // point to the end of the data segment.
-  ElfSym<ELFT>::End.st_value = VA;
-
   for (Phdr &PHdr : Phdrs) {
     Elf_Phdr &H = PHdr.H;
     if (PHdr.First) {
       OutputSectionBase<ELFT> *Last = PHdr.Last;
       H.p_filesz = Last->getFileOff() - PHdr.First->getFileOff();
       if (Last->getType() != SHT_NOBITS)
         H.p_filesz += Last->getSize();
       H.p_memsz = Last->getVA() + Last->getSize() - PHdr.First->getVA();
template <class ELFT> void Writer<ELFT>::fixAbsoluteSymbols() {
   }
 
   // Update MIPS _gp absolute symbol so that it points to the static data.
   if (Config->EMachine == EM_MIPS)
     ElfSym<ELFT>::MipsGp.st_value = getMipsGpAddr<ELFT>();
 
   // _etext is the first location after the last read-only loadable segment.
   // _edata is the first location after the last read-write loadable segment.
+  // _end is the first location after the uninitialized data region.
   for (Phdr &PHdr : Phdrs) {
     if (PHdr.H.p_type != PT_LOAD)
       continue;
+    ElfSym<ELFT>::End.st_value = PHdr.H.p_vaddr + PHdr.H.p_memsz;
     uintX_t Val = PHdr.H.p_vaddr + PHdr.H.p_filesz;
     if (PHdr.H.p_flags & PF_W)
       ElfSym<ELFT>::Edata.st_value = Val;
     else
       ElfSym<ELFT>::Etext.st_value = Val;
   }
 }