[llvm] r207693 - Revert r207571 - Add slp vectorization to LTO passes

Eric Christopher echristo at gmail.com
Wed Apr 30 12:40:51 PDT 2014


In the future please explain why in the commit message.

So, as a followup, why are you reverting?

-eric

On Wed, Apr 30, 2014 at 12:27 PM, Yi Jiang <yjiang at apple.com> wrote:
> Author: yjiang
> Date: Wed Apr 30 14:27:24 2014
> New Revision: 207693
>
> URL: http://llvm.org/viewvc/llvm-project?rev=207693&view=rev
> Log:
> Revert r207571 - Add slp vectorization to LTO passes
>
> Modified:
>     llvm/trunk/lib/Transforms/IPO/PassManagerBuilder.cpp
>
> Modified: llvm/trunk/lib/Transforms/IPO/PassManagerBuilder.cpp
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Transforms/IPO/PassManagerBuilder.cpp?rev=207693&r1=207692&r2=207693&view=diff
> ==============================================================================
> --- llvm/trunk/lib/Transforms/IPO/PassManagerBuilder.cpp (original)
> +++ llvm/trunk/lib/Transforms/IPO/PassManagerBuilder.cpp Wed Apr 30 14:27:24 2014
> @@ -341,9 +341,6 @@ void PassManagerBuilder::populateLTOPass
>    PM.add(createLoopDeletionPass());
>    PM.add(createLoopVectorizePass(true, true));
>
> -  // More scalar chains could be vectorized due to more alias information
> -  PM.add(createSLPVectorizerPass()); // Vectorize parallel scalar chains.
> -
>    // Cleanup and simplify the code after the scalar optimizations.
>    PM.add(createInstructionCombiningPass());
>
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits



More information about the llvm-commits mailing list