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

Brian Rzycki via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 12 07:53:54 PST 2018


brzycki added a comment.

> 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.

Don't spend too much time on it if you're confident it's not an issue. I was just curious in case you had already run the numbers.  PDT is used less than DT all around the middle end which is also why I suspect you haven't seen this issue in 6.0.


Repository:
  rL LLVM

https://reviews.llvm.org/D43140





More information about the llvm-commits mailing list