[PATCH] D29330: Replace MergeOutputSection with synthetic input section MergeSection.

Rui Ueyama via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 1 17:38:12 PST 2017


ruiu accepted this revision.
ruiu added a comment.

LGTM



================
Comment at: ELF/Writer.cpp:1025
 static void removeUnusedSyntheticSections(std::vector<OutputSectionBase *> &V) {
-  // Input synthetic sections are placed after all regular ones. We iterate over
-  // them all and exit at first non-synthetic.
+  // All Input synthetic sections that can be empty are placed after
+  // all regular ones. We iterate over them all and exit at first
----------------
s/All Input/All input/


https://reviews.llvm.org/D29330





More information about the llvm-commits mailing list