[PATCH] D13911: [ELF2] Determine the order of entries of symbol tables in the finalize() phase.

Rui Ueyama via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 20 14:20:08 PDT 2015


ruiu accepted this revision.
ruiu added a comment.
This revision is now accepted and ready to land.

LGTM with a nit.


================
Comment at: ELF/OutputSections.cpp:698
@@ +697,3 @@
+      Body->setDynamicSymbolTableIndex(++I);
+  } else {
+    std::stable_sort(Symbols.begin(), Symbols.end(),
----------------
Early return before this 'else' to remove this 'else'.


http://reviews.llvm.org/D13911





More information about the llvm-commits mailing list