[PATCH] D17529: ELF: Implement ICF.

[Sean Silva via llvm-commits]

silvas added a comment.

Some tiny wordsmithing nits.


================
Comment at: ELF/ICF.cpp:168
@@ +167,3 @@
+                          Comparator Eq) {
+  // This loop sorts [Begin, End) so that all sections that are equal
+  // in terms of Eq are contiguous. The algorithm is quadratic
----------------
I would say "rearranges" instead of "sorts", since "sort" implies an absolute order.

================
Comment at: ELF/ICF.cpp:169
@@ +168,3 @@
+  // This loop sorts [Begin, End) so that all sections that are equal
+  // in terms of Eq are contiguous. The algorithm is quadratic
+  // in the worse case, but that is not an issue in practice because
----------------
I think it is worth clarifying that it is quadratic in the number of *distinct* sections.


http://reviews.llvm.org/D17529