Page MenuHomePhabricator

IPO, LTO: Plumb the summary from the LTO API into the pass manager.
ClosedPublic

Authored by pcc on Jan 17 2017, 5:56 PM.

Diff Detail

Repository
rL LLVM

Event Timeline

pcc created this revision.Jan 17 2017, 5:56 PM
mehdi_amini accepted this revision.Jan 17 2017, 10:28 PM

LGTM.

llvm/include/llvm/LTO/LTOBackend.h
38 ↗(On Diff #84788)

May not be straightforward to know why "a regular LTO backend" need a "CombinedIndex", so worth commenting.

llvm/include/llvm/Transforms/IPO/PassManagerBuilder.h
128 ↗(On Diff #84788)

I'd add , for example the CFI and devirtualization type tests.

llvm/lib/LTO/LTOBackend.cpp
199 ↗(On Diff #84788)

FIXME?

This revision is now accepted and ready to land.Jan 17 2017, 10:28 PM
This revision was automatically updated to reflect the committed changes.
pcc marked 3 inline comments as done.