[PATCH] D39406: Merge SymbolBody and Symbol into one class, SymbolBody.

[Rui Ueyama via Phabricator via llvm-commits]

ruiu added a comment.

>> If we have to pack it better, we can do it by tweaking some members of SymbolBody, but looks like the program isn't very sensitive to the size of Symbol.
> 
> If that's what you've measured, this seems fine to me.

Yes, that's what I measured. I tried to link clang without debug info (w/debug wasn't a good test because it spent most of its execution time to merge strings), and the difference before and after was a noise.


https://reviews.llvm.org/D39406