[PATCH] D134178: [SCEV] Calculate PHI range based on its underlying values instead of just its inputs

[Nikita Popov via Phabricator via llvm-commits]

nikic added a comment.

In D134178#3799419 <https://reviews.llvm.org/D134178#3799419>, @mkazantsev wrote:

> @nikic this is not a final solution, but could you please evaluate CT impact of this one? Just for reference.

Unfortunately there is a crash when building test-suite: http://llvm-compile-time-tracker.com/show_error.php?commit=d1ebd749bf430527616aab33a7340f9fd2736228

Note that to make this change viable, every pass that preserves SCEV must also preserve PhiValues -- I doubt this is worth the effort or complexity.


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

https://reviews.llvm.org/D134178