[PATCH] D75362: [InstCombine] Process blocks in RPO

[Nikita Popov via Phabricator via llvm-commits]

nikic added a comment.

In D75362#1898366 <https://reviews.llvm.org/D75362#1898366>, @xbolva00 wrote:

> Did you observe any (positive) compile time changes?


It turned out that computing the RPO order is more expensive than I expected... After optimizing a bit, this change still clocks in as a 0.15% regression <http://llvm-compile-time-tracker.com/compare.php?from=6ba63510720f5bd6033a3eeffd51f8d7e0e90432&to=d8655910e8b5764a54ba5c5d110a9d87eb89aa1f&stat=instructions>.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D75362/new/

https://reviews.llvm.org/D75362