[PATCH] D48000: [InstSimplify]update simplifyUnsignedRangeCheck function's test case.
Sanjay Patel via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Jun 12 08:12:39 PDT 2018
spatel added a comment.
In https://reviews.llvm.org/D48000#1129208, @HLJ2009 wrote:
> In https://reviews.llvm.org/D48000#1128454, @spatel wrote:
>
> > I think we've sorted out the problem in https://reviews.llvm.org/D47922, but this patch is not correct as shown.
> >
> > It should show the results (the missed optimization) that we currently get using trunk. Please update.
>
>
> Can you help me look at my submission? I use the trunk branch code.
In this patch, you want to add new tests to trunk (the code change from https://reviews.llvm.org/D47922 should not exist).
Run utils/update_test_checks.py with these new tests. There should be at least one missed optimization in the CHECK lines (because https://reviews.llvm.org/D47922 is not applied).
Repository:
rL LLVM
https://reviews.llvm.org/D48000
More information about the llvm-commits
mailing list