[PATCH] D18480: Reapply [LSR] Create fewer redundant instructions.

[Quentin Colombet via llvm-commits]

Hi Geoff,

> On Mar 29, 2016, at 10:53 AM, Geoff Berry <gberry at codeaurora.org> wrote:
> 
> gberry added a comment.
> 
> Thanks Quentin, but this patch exposes what I believe is a bug in the interaction between LSR and SCEV.  I'm still in the process of resolving that issue.
> If you're interested, the relevant email thread on llvm-dev has the Subject line "LSR/SCEV problem/question”

I thought you already fixed it and thought that, since this patch was exactly the same as the one I reviewed the last time, it was a side effect exposed by this patch but not a problem with this patch.

Let me look closer to see if I can help then.

Cheers,
-Quentin



> 
> 
> http://reviews.llvm.org/D18480
> 
> 
>