[PATCH] D42442: [ValueTracking] add an assert to prevent infinite recursion

[Sanjay Patel via Phabricator via llvm-commits]

spatel added a comment.

In https://reviews.llvm.org/D42442#985767, @efriedma wrote:

> > As I understand it, we want to uncover these bugs in passes that use value tracking because it signals a potential efficiency improvement for them when dealing with dead code or other corner cases.
>
> We don't actually have a solution to reliably avoid analyzing dead code... see the discussion on https://reviews.llvm.org/D34135.


Ah...so should we just bail out here in value tracking then?


https://reviews.llvm.org/D42442