[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