Ping: Add a Scalarizer pass

Pekka Jääskeläinen pekka.jaaskelainen at tut.fi
Wed Nov 20 03:22:58 PST 2013


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



More information about the llvm-commits mailing list