[PATCH] D19528: [ELF] - Implemented -z combrelocs/nocombreloc.

[Rafael Ávila de Espíndola via llvm-commits]

rafael added a comment.

Please do simplify the patch to first not set any new DT_ and sort only by offset. Lets benchmark just that first.


================
Comment at: ELF/Writer.cpp:168
@@ -166,3 +167,3 @@
     GotPlt.reset(new GotPltSection<ELFT>);
-    RelaPlt.reset(new RelocationSection<ELFT>(S));
+    RelaPlt.reset(new RelocationSection<ELFT>(S, false /*Sort*/));
   }
----------------
Why can't you sort this? It shouldn't hurt when being lazy and it should help otherwise.


http://reviews.llvm.org/D19528