[PATCH] D18960: [lld] Initial implementation of TLSDESC relocation handling

[Ed Maste via llvm-commits]

emaste added inline comments.

================
Comment at: ELF/OutputSections.h:297
@@ -287,2 +296,3 @@
   std::vector<DynamicReloc<ELFT>> Relocs;
+  std::vector<DynamicReloc<ELFT>> TlsDescRelocs;
 };
----------------
rafael wrote:
> Why do you need the second vector? It seems you just relocate it at the same time as Relocs.
The TLSDESC relocs have to follow the others; earlier versions of the patch kept track of the number of different types and inserted at the appropriate point in the vector (and a middle version had a vector of reloc vectors), but this way seems more clear to me.


http://reviews.llvm.org/D18960