[llvm] r239115 - [InstCombine] Don't miscompile safe increment idiom

[Daniel Jasper]

I have verified that this is the root cause. From a brief look, I'd guess
that the return needs to go into the "if (Simplified) {...}" block, but I
don't understand this enough. I'll revert this for now so that David can
take a look.

On Fri, Jun 5, 2015 at 10:25 AM, Renato Golin <renato.golin at linaro.org>
wrote:

> On 5 June 2015 at 00:11, David Majnemer <david.majnemer at gmail.com> wrote:
> > Author: majnemer
> > Date: Thu Jun  4 18:11:30 2015
> > New Revision: 239115
> >
> > URL: http://llvm.org/viewvc/llvm-project?rev=239115&view=rev
> > Log:
> > [InstCombine] Don't miscompile safe increment idiom
>
> Hi David,
>
> I haven't bisected yet, but it's likely that this commit is the one
> responsible for this failure:
>
>
> http://lab.llvm.org:8011/builders/clang-x86_64-linux-selfhost-abi-test/builds/5543
>
> And all self-hosting + test-suite failures in ARM. :)
>
> The effect is the same, times out, gets killed.
>
> cheers,
> --renato
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150605/3632c37e/attachment.html>