[PATCH] D38586: SLPVectorizer.cpp: Avoid std::stable_sort() due to asymmetric comparator

Davide Italiano via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 5 15:04:19 PDT 2017


davide added a comment.

In https://reviews.llvm.org/D38586#889903, @dberlin wrote:

> We sort siblings by RPO in newgvn. If you want to guarantee parent before child, however, just sort by the dominator tree dfs in and out numbers.


+1, yes, that's where I got the idea from :)


Repository:
  rL LLVM

https://reviews.llvm.org/D38586





More information about the llvm-commits mailing list