[PATCH] Extend SLPVectorizer for cases where insertelement instructions must be rescheduled

[Nadav Rotem]

On Mar 27, 2014, at 9:55 PM, Arnold Schwaighofer <aschwaighofer at apple.com> wrote:

> How about we simplify the logic in BoUpSLP by sinking the logic for handling the “BuildVector” into the SLPVectorizer like in the attached patch?
> 
> This reduces the interface of BoUpSLP to just a “UserIgnoreList” that is used when checking scheduling and generation of extracts for out of tree users.
> 
> Nadav, is this in the spirit of what you meant today?

Arnold, I really like your approach. Thanks!

> 
> <0001-SLPVectorizer-Ignore-users-that-are-insertelements-w.patch>
> On Mar 27, 2014, at 10:38 AM, Arch D. Robison <arch.robison at intel.com> wrote:
> 
>> 
>> Thanks for the suggestions.  the find-->count change will eliminate the long line issue :-)
>> 
>> I didn't understand the suggestion of changing:
>> 
>>     IE->removeFromParent();
>>     IE->insertAfter(x);`
>> 
>> to:
>> 
>>     IE->moveBefore(x)
>> 
>> Aren't these different?  I couldn't find a "moveAfter" method.
>> 
>> http://llvm-reviews.chandlerc.com/D3143
>