[PATCH] D22377: [SCEV] trip count calculation for loops with unknown stride

[Sanjoy Das via llvm-commits]

On Fri, Sep 2, 2016 at 1:35 PM, Pankaj Chawla <pankaj.chawla at intel.com> wrote:
> pankajchawla added a comment.
>
> Hi all,
>
> I have a fix for the issue. The max backedge taken count was incorrectly set to 1 for loops with unknown strides which resulted in LSR perofming the wrong transformation. I verified that we generate identical assembly code for the reproducer without my patch and with my patch + additional fix.
>
> It looks like I am not able to upload the diff to this review since it is closed. Do I have to open a new review?

Opening a new review SGTM.

-- Sanjoy

>
> Thanks,
> Pankaj
>
>
> Repository:
>   rL LLVM
>
> https://reviews.llvm.org/D22377
>
>
>



-- 
Sanjoy Das
http://playingwithpointers.com