[PATCH] D19953: ELF: Do not use -1 to mark pieces of merge sections as being tail merged.

[Rui Ueyama via llvm-commits]

ruiu added inline comments.

================
Comment at: ELF/Writer.cpp:1127-1129
@@ -1123,2 +1126,5 @@
       return false;
+    if (auto *S = dyn_cast<MergeInputSection<ELFT>>(D->Section))
+      if (S->getRangeAndSize(D->Value).first->second == typename ELFT::uint(-1))
+        return false;
   }
----------------
I may be missing something, but I don't think we gc pieces of mergeable sections. Instead, we gc a mergeable sections as a whole -- all the pieces in a mergeable section are gc'ed or not. So do you need this test?


http://reviews.llvm.org/D19953