[llvm] r207693 - Revert r207571 - Add slp vectorization to LTO passes
yijiang
yjiang at apple.com
Wed Apr 30 12:51:15 PDT 2014
yes, I should have mentioned that the reason is that this will trigger eon’s execution fail.
On Apr 30, 2014, at 12:40 PM, Eric Christopher <echristo at gmail.com> wrote:
> 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