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

[Mehdi AMINI via Phabricator via cfe-commits]

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.




https://reviews.llvm.org/D34156