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

Amaury SECHET via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 25 23:37:58 PDT 2015


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

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D12269.33182.patch
Type: text/x-patch
Size: 11066 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150826/05fe5456/attachment.bin>


More information about the llvm-commits mailing list