[lld] [lld][COFF] Remove duplicate strtab entries (PR #141197)

[Martin Storsjö via llvm-commits]

mstorsjo wrote:

> How about using two StringTableBuilder, one for long section name and another for others? I have observed large strtab size many times for large projects. Will you revive your patch with supporting for prioritized long section name?

I don't plan on reviving it myself right now, but if you're working in this area, I would appreciate if you'd try to reapply that patch (revert the revert) and see if it can be made to work again on current git main. Then with that in place, it would be interesting to see if you see any further significant size reductions on your blender testcase.

If it doesn't really reduce the binary size any much further (perhaps cases where tail merging makes any difference are rare?), then perhaps it's best to just go with the much simpler approach of your current patch here.

https://github.com/llvm/llvm-project/pull/141197