[PATCH] D33584: Remove a quadratic behavior in assert-enabled builds

Daniel Berlin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri May 26 08:14:53 PDT 2017


dberlin added a comment.

As I said last year, i believe, we should just remove this assert.
It doesn't help anything. The callers literally  can't  handle it any better if they want real dominance answers.
There isn't anywhere else in llvm we assert because "a thing may become quadratic if you do dumb things", and the assert itself is quadratic.


https://reviews.llvm.org/D33584





More information about the llvm-commits mailing list