Page MenuHomePhabricator

Please use GitHub pull requests for new patches. Avoid migrating existing patches. Phabricator shutdown timeline

Differential D19399

Third patch for inlining report
Needs ReviewPublic
Actions

Authored by rcox2 on Apr 21 2016, 6:40 PM.

Details

Reviewers
hfinkel
apilipenko
reames
Summary

Third patch for inlining report

This is the third patch out of 3 for the inlining report. It ties together the first patch, which provided basic functionality and the second patch, which added info to the cost files to create inlining reasons.

Diff Detail

Event Timeline

rcox2 updated this revision to Diff 54602.Apr 21 2016, 6:40 PM
rcox2 retitled this revision from to Third patch for inlining report .
rcox2 updated this object.
rcox2 added reviewers: reames, hfinkel, apilipenko.
rcox2 added subscribers: kbsmith1, llvm-commits.
davidxl added subscribers: davidxl, eraman.Apr 21 2016, 9:16 PM
davidxl added a comment.Apr 22 2016, 9:57 AM

Robert, can you update your patch with more context lines?

reames resigned from this revision.Feb 25 2020, 9:01 AM

Resigning from a stale review (2016). Feel free to re-add if thread ever revived.

Revision Contents
Changeset List

PathSize
include/
llvm/
Transforms/
IPO/
InlineReport.h
InlineReport.h
33 lines
Utils/
11 lines
lib/
Transforms/
IPO/
InlineAlways.cpp
InlineAlways.cpp
4 lines
InlineReport.cpp
InlineReport.cpp
333 lines
Inliner.cpp
Inliner.cpp
42 lines
Utils/
65 lines

Diff 54602

View Options

include/llvm/Transforms/IPO/InlineReport.h

Loading...
View Options

include/llvm/Transforms/Utils/Cloning.h

Loading...
View Options

lib/Transforms/IPO/InlineAlways.cpp

Loading...
View Options

lib/Transforms/IPO/InlineReport.cpp

Loading...
View Options

lib/Transforms/IPO/Inliner.cpp

Loading...
View Options

lib/Transforms/Utils/InlineFunction.cpp

Loading...