[llvm] r280637 - [InstCombine] allow icmp (and X, C2), C1 folds for splat constant vectors

Sanjay Patel via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 5 15:49:24 PDT 2016


Hi Renato -

Thanks for letting me know. Feel free to revert anything I've done if it's
a suspect. I've reverted this commit at r280676 and will investigate.

PS - I'm "Sanjay". :)

On Mon, Sep 5, 2016 at 4:11 PM, Renato Golin <renato.golin at linaro.org>
wrote:

> On 4 September 2016 at 21:58, Sanjay Patel via llvm-commits
> <llvm-commits at lists.llvm.org> wrote:
> > Author: spatel
> > Date: Sun Sep  4 15:58:27 2016
> > New Revision: 280637
> >
> > URL: http://llvm.org/viewvc/llvm-project?rev=280637&view=rev
> > Log:
> > [InstCombine] allow icmp (and X, C2), C1 folds for splat constant vectors
> >
> > The code to calculate 'UsesRemoved' could be simplified.
> > As-is, that code is a victim of PR30273:
> > https://llvm.org/bugs/show_bug.cgi?id=30273
>
> Hi Sanjoy,
>
> This one is probably yours:
>
> http://lab.llvm.org:8011/builders/clang-cmake-armv7-
> a15/builds/14952/steps/ninja%20check%201/logs/FAIL%3A%20LLVM%3A%3Aicmp.ll
>
> Unfortunately, our bots didn't detect because of a change in the
> builder which made all bots not report any errors.
>
> If you can't fix it quickly, please revert and we'll investigate that
> later, as right now, we're dealing with a large number of concurrent
> failures and we won't have time to investigate further.
>
> Sorry.
>
> --renato
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160905/5dd7d81c/attachment.html>


More information about the llvm-commits mailing list