[PATCH] D26634: Make block placement determinisatic

Benjamin Kramer via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 16 04:43:31 PST 2016


bkramer added a comment.

Note that SetVector::remove is O(n) whereas SmallPtrSet::erase was O(1). I guess it doesn't matter too much, land this now to make codegen deterministic again. If perf issues arise we can come up with a more clever way (e.g. sorting by block number before iterating over the set).


https://reviews.llvm.org/D26634





More information about the llvm-commits mailing list