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

Davide Italiano via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 1 10:21:05 PST 2017

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.



More information about the llvm-commits mailing list