[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.
https://reviews.llvm.org/D34156
More information about the cfe-commits
mailing list