[PATCH] D60846: [ValueTracking] Improve isKnowNonZero for Ints

Dan Robertson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Apr 28 17:41:34 PDT 2019


dlrobertson added a comment.

In D60846#1481782 <https://reviews.llvm.org/D60846#1481782>, @spatel wrote:

> In D60846#1481774 <https://reviews.llvm.org/D60846#1481774>, @nikic wrote:
>
> > test/Transforms/LICM/hoist-mustexec.ll still needs to be updated. (Should probably generate full checks as a preliminary NFC commit first.)
>
>
> Oops - I forgot that even though I mentioned it in the earlier comment. Yes, that must be included in this patch (and any other existing tests that are known to be affected).


Sorry, I forgot all about this as well. I did run the `update_test_checks` script on this file after applying the patch and there was not improvement to the generated code.


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

https://reviews.llvm.org/D60846





More information about the llvm-commits mailing list