Ping: Add a Scalarizer pass

Bill Wendling isanbard at gmail.com
Wed Nov 20 21:24:55 PST 2013


Sorry, this is a new pass, so I’m not really comfortable adding it to 3.4, even if it’s not used.

-bw

On Nov 20, 2013, at 3:22 AM, Pekka Jääskeläinen <pekka.jaaskelainen at tut.fi> wrote:

> LGTM.
> 
> IMHO this could be committed to the 3.4 branch as it's not part
> of the default pass chain.
> 
> On 11/20/2013 01:06 PM, Richard Sandiford wrote:
>> Ping for this update, which I think addresses all review comments so far.
>> 
>> Thanks,
>> Richard
>> 
>>> Remove use of TargetTransformInfo.  Do not add the pass to PassManagerBuilder.
>>> 
>>> We also talked about whether to use a reverse post-order traversal,
>>> but I don't think it's worth it in the use cases I have.  It could
>>> easily be added later it we find a counterexample though.
>> 
>> http://llvm-reviews.chandlerc.com/D2112
>> 
>> 
>> 
>> _______________________________________________
>> llvm-commits mailing list
>> llvm-commits at cs.uiuc.edu
>> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>> 
> 
> 
> -- 
> Pekka
> _______________________________________________
> 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