[PATCH] D43140: [Dominators] Always recalculate postdominators when update yields different roots

Jakub Kuderski via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 12 07:50:18 PST 2018


kuhar added a comment.

In https://reviews.llvm.org/D43140#1005098, @brzycki wrote:

> LGTM. Out of curiosity does this change impact compile time?


I haven't checked, but from my previous experiments I doubt it does -- forming or to infinite loops are very rare, and this piece of code was only executed a few times when compiling clang or sqlite. I don't have numbers for it, but could get them later this week if someone's really interested.


Repository:
  rL LLVM

https://reviews.llvm.org/D43140





More information about the llvm-commits mailing list