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
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
