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

[Marianne Mailhot-Sarrasin]

Thanks. Could you please commit it ?

Marianne

From: Nadav Rotem [mailto:nrotem at apple.com]
Sent: June-22-15 6:25 PM
To: Marianne Mailhot-Sarrasin
Cc: llvm-commits at cs.uiuc.edu
Subject: Re: [PATCH] [LoopVectorize] Detect correctly if vectorizer will create a tail loop

Looks good to me.


On Jun 22, 2015, at 12:17 PM, Marianne Mailhot-Sarrasin <marianne.mailhot-sarrasin at octasic.com<mailto:marianne.mailhot-sarrasin at octasic.com>> wrote:

I have added it in the patch.

From: Nadav Rotem [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/20150623/e412347c/attachment.html>