[PATCH] [LoopVectorize] Detect correctly if vectorizer will create a tail loop

[Nadav Rotem]

Looks good to me. 


> On Jun 22, 2015, at 12:17 PM, Marianne Mailhot-Sarrasin <marianne.mailhot-sarrasin at octasic.com> wrote:
> 
> I have added it in the patch.
>  
> From: Nadav Rotem [mailto:nrotem at apple.com <mailto:nrotem at apple.com>] 
> Sent: June-22-15 1:21 PM
> To: Marianne Mailhot-Sarrasin
> Cc: llvm-commits at cs.uiuc.edu <mailto:llvm-commits at cs.uiuc.edu>
> Subject: Re: [PATCH] [LoopVectorize] Detect correctly if vectorizer will create a tail loop
>  
> Hi Marianne, 
>  
> Can you please provide a testcase for this fix?
>  
> Thanks,
> Nadav
>  
>  
> On Jun 22, 2015, at 7:01 AM, Marianne Mailhot-Sarrasin <marianne.mailhot-sarrasin at octasic.com <mailto:marianne.mailhot-sarrasin at octasic.com>> wrote:
>  
> Hi,
>  
> Here is a simple bug fix. With option OptForSize enabled, the Loop Vectorizer is not supposed to create tail loop. The condition checking that was invalid and was not matching to the comment above.
>  
> Could someone please review and commit if valid ?
>  
> Thanks,
> Marianne
> <LoopVectorize.patch>
>  
> <LoopVectorize.patch>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150622/2f601c7c/attachment.html>