[PATCH] D36963: Use sorting instead of a lot of stable_partitions for `ICF::segregate` function

[Rui Ueyama via Phabricator via llvm-commits]

ruiu added a comment.

You may be right, but I'm still trying to figure out the source of nondeterminism. I disabled /DEBUG and ran the same command with and without this patch, and I'm still seeing that only the case with this patch produces nondeterministic outputs (outputs are different in size).

If I replaced parallel::par with parallel:seq in this file, it became deterministic, so there must be a threading issue, but I can't figure out the cause. It is pretty tricky.


https://reviews.llvm.org/D36963