[PATCH] D28129: NewGVN: Sort Dominator Tree in RPO order, and use that for generating order.

Davide Italiano via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 28 07:05:23 PST 2016


davide added a comment.

In https://reviews.llvm.org/D28129#631695, @davide wrote:

> Sorry for the slow response, I'm out('ish) of the office these days. I took a close look at your patch.
>  I happen to be lucky enough to hit a case in the wild where this already matters.  The number of iteration goes down from hundreds to ~10, which makes compile time/me happier. 
>  I'll see if I can investigate


*I'll see if I can investigate the use of this tecnique in other passes in the future.


https://reviews.llvm.org/D28129





More information about the llvm-commits mailing list