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

deadal nix via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 27 13:13:21 PDT 2015


Chandler, any news ? Don't be afraid to do a raw brain dump, that would at
least give some matter to get started.

2015-08-26 2:12 GMT-07:00 Chandler Carruth <chandlerc at gmail.com>:

> 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/20150827/65785449/attachment.html>


More information about the llvm-commits mailing list