[PATCH] D29395: [ValueTracking] avoid crashing from bad assumptions (PR31809)

[Davide Italiano via Phabricator via llvm-commits]

davide added a comment.

In https://reviews.llvm.org/D29395#663457, @spatel wrote:

> F3051803: badass2.patch <https://reviews.llvm.org/F3051803>
>
> Attaching a draft of the assumption cache invalidation patch that I was working on. This is just for reference. I don't think we want to pursue it because it's more trouble than it's worth even if it is "stronger".


I don't think it's stronger, but it's definitely more trouble. Remove the FIXME from the original patch and LGTM.


Repository:
  rL LLVM

https://reviews.llvm.org/D29395