[PATCH] D44504: COFF: Implement string tail merging.

[Peter Collingbourne via Phabricator via llvm-commits]

pcc added a comment.

I will commit when https://reviews.llvm.org/D44501 is approved :)



================
Comment at: lld/COFF/ICF.cpp:230
+  for (auto &P : MergeChunk::Instances)
+    for (auto *SC : P.second->Sections)
+      SC->Class[0] = NextId++;
----------------
ruiu wrote:
> pcc wrote:
> > ruiu wrote:
> > > auto -> MergeSection
> > It would actually be `std::pair<const uint32_t, MergeChunk *>`, which seems a little verbose, so I left it the way it is.
> But what about SC? Isn't it SectionChunk?
Oh, right.


https://reviews.llvm.org/D44504