[PATCH] D37467: Add a new pass to speculate around PHI nodes with constant (integer) operands when profitable.

[Chandler Carruth via Phabricator via llvm-commits]

chandlerc added a comment.

FWIW, I think with the latest patch update this has now addressed Danny's primary concerns, and should be much more effective at avoiding re-visiting regions of IR.

The biggest remaining issue of revisiting regions of IR is the fact that we don't cache things between basic blocks. We could move to do that, but I'm worried about how much more complex the logic would become. Would appreciate thoughts on that from others.


https://reviews.llvm.org/D37467