[PATCH] D34156: [LTO] Add -femit-summary-index flag to emit summary for regular LTO

Mehdi AMINI via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jun 14 14:30:28 PDT 2017

mehdi_amini added a comment.

In https://reviews.llvm.org/D34156#780570, @tobiasvk wrote:

> - Change patch to always emit a module summary for regular LTO
>   I don't see any real downside of having a summary given the marginal time and space overhead it takes to construct and save it.

I think some code/logic needs to be changed in LLVM first:

  bool LTOModule::isThinLTO() {
    // Right now the detection is only based on the summary presence. We may want
    // to add a dedicated flag at some point.


More information about the cfe-commits mailing list