[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