iains added a comment. In D144844#4195316 <https://reviews.llvm.org/D144844#4195316>, @ChuanqiXu wrote:
> Got your points. Let's postpone this one. > > But I want to emphasize that this patch (and the thin PCM) will decrease the > performance. While LTO can save the regression, LTO is not widely used. > (ThinLTO can only mitigate this.) I mean the default behavior shouldn't cause > performance regression. We can offer a new alternative for the users but we > can't enable that by default simply now. Agree that we (ideally, at least) should not decrease performance with a new feature. However, "performance" also includes compilation speed in the 'no optimisation, debug' case - that is also considered very important. So, perhaps, the short-term approach should be (as @dblaikie suggested) to include the bodies for `-O` >= 3? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D144844/new/ https://reviews.llvm.org/D144844 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
