[llvm] r190916 - Lift alignment restrictions for load/store folding on VINSERTF128/VEXTRACTF128. Fixes PR17268. [PATCH]

[Pekka Jääskeläinen]

Hi,

The attached patch reverts the line that forces the SLPVectorizer
always on in opt. Passes "make check" in LLVM 3.4.

OK to commit?

On 11/19/2013 09:51 AM, Nick Lewycky wrote:
> On 17 September 2013 20:55, Craig Topper <craig.topper at gmail.com
> <mailto:craig.topper at gmail.com>> wrote:
>
>     Author: ctopper
>     Date: Tue Sep 17 22:55:53 2013
>     New Revision: 190916
>
>     URL: http://llvm.org/viewvc/llvm-project?rev=190916&view=rev
>     Log:
>     Lift alignment restrictions for load/store folding on
>     VINSERTF128/VEXTRACTF128. Fixes PR17268.
>
>
>     Modified: llvm/trunk/tools/opt/opt.cpp
>     URL:
>     http://llvm.org/viewvc/llvm-project/llvm/trunk/tools/opt/opt.cpp?rev=190916&r1=190915&r2=190916&view=diff
>     ==============================================================================
>     --- llvm/trunk/tools/opt/opt.cpp (original)
>     +++ llvm/trunk/tools/opt/opt.cpp Tue Sep 17 22:55:53 2013
>     @@ -462,6 +462,7 @@ static void AddOptimizationPasses(PassMa
>                                      DisableLoopUnrolling : OptLevel == 0;
>
>         Builder.LoopVectorize = OptLevel > 1 && SizeLevel < 2;
>     +  Builder.SLPVectorize = true;
>
>
> This doesn't match the commit log. Was this intentional?
>
> Nick

-- 
Pekka
-------------- next part --------------
A non-text attachment was scrubbed...
Name: do-not-force-slp-vectorize-in-opt.patch
Type: text/x-patch
Size: 442 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20131121/168d367f/attachment.bin>