[PATCH] D12269: Add a pass to lift aggregate into allocas so SROA can get rid of them.

Chandler Carruth via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 26 02:12:17 PDT 2015


I'm really not sure this is the right approach, despite liking a number of
aspects and being optimistic early-on. But I've not yet had time to put my
thoughts here into a nice cohesive email. I'll do that as soon as I can to
help resolve this.

On Tue, Aug 25, 2015 at 11:38 PM Amaury SECHET via llvm-commits <
llvm-commits at lists.llvm.org> wrote:

> deadalnix updated this revision to Diff 33182.
> deadalnix removed a reviewer: chandlerc.
> deadalnix added a comment.
>
> Various nits.
> Use FunctionLifter as a per function state.
>
> Is the approach agreed upon ? If so, what is the best place to put this in
> ?
> @chandlerc You seems to me the SROA guy, do you think this could be
> integrated with in SROA ? If yes, what is the best way to proceed ? If no,
> what is a good place to put this in ?
>
>
> http://reviews.llvm.org/D12269
>
> Files:
>   include/llvm/InitializePasses.h
>   include/llvm/Transforms/Scalar.h
>   lib/Transforms/IPO/PassManagerBuilder.cpp
>   lib/Transforms/Scalar/AggregateLifter.cpp
>   lib/Transforms/Scalar/CMakeLists.txt
>   lib/Transforms/Scalar/Scalar.cpp
>   test/Transforms/AggregateLifter/extractvalue.ll
>   test/Transforms/AggregateLifter/store.ll
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150826/2f290f12/attachment.html>


More information about the llvm-commits mailing list