[PATCH] D51443: [dsymutil] Remove non-determinism.

[Frederic Riss via Phabricator via llvm-commits]

friss added a comment.

In https://reviews.llvm.org/D51443#1225644, @JDevlieghere wrote:

> dSYM for a clang modules build:
>
>   $ stat -f%z tot/Contents/Resources/DWARF/clang
>   962974260
>  
>   $ stat -f%z differential/Contents/Resources/DWARF/clang
>   962974305
>
>
> I understand the difference would be larger for an Objective-C project, so I'll try to benchmark that as well.


It doesn't hurt to check, but the difference should be smaller when using Obj-C modules. The reason is that for ObjC modules everything will be pruned in the skeleton CUs, everything gets uniqued to the module debug info at the beginning. With C++, template instantiations are not in the module debug info, so they could be uniqued afterwards (and won't with this change). I'm actually surprised that the difference is not bigger for clang.


https://reviews.llvm.org/D51443