[PATCH] D23149: Printing imported functions size

Mehdi AMINI via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 4 16:34:10 PDT 2016


mehdi_amini added a comment.

In https://reviews.llvm.org/D23149#506379, @tejohnson wrote:

> Right, I was originally thinking about suggesting an alternative which is to pass the index into the inliner so the info could simply be queried directly out of the index. However, it makes dumping these stats via something like "llvm-link -thinlto-action=import | opt -O2" impossible unless we pass the combined index in to opt as well, for passing to the inliner.


Could be a new optional `cl::opt` for `ImportedFunctionsInliningStatistics` providing the path to the index? 
Otherwise it can also be that the printed statistics have to be passed to a third party tool to produce a report (with a flow similar to how code-coverage reporting works somehow).


https://reviews.llvm.org/D23149





More information about the llvm-commits mailing list