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

In https://reviews.llvm.org/D37467#936896, @MatzeB wrote:

> LGTM
>
> > I've also only implemented this in the new pass manager. If folks are
>
> very interested, I can try to add it to the old PM as well, but I didn't
>  really see much point (my use case is already switched over to the new
>  PM).
>
> Isn't the legacy PM still the default we use for most frontends including clang and the new PM is only used for (Thin)LTO so far?


Clang has a flag to use the new PM everywhere. Google is already using it by default, and I have an email thread on llvm-dev about the (very few) things left before we can make it the default for Clang.


https://reviews.llvm.org/D37467