[PATCH] D153701: [WIP][Clang] Implement P2718R0 "Lifetime extension in range-based for loops"
Hubert Tong via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Aug 9 17:04:15 PDT 2023
hubert.reinterpretcast added a comment.
In D153701#4563919 <https://reviews.llvm.org/D153701#4563919>, @yronglin wrote:
> The gap between these two numbers is very large. So I'think we can create additional materializations only within for-range initializers. I'm not sure if I can find a way to only create materializes for temporaries that need to have an extended lifetime, WDYT?
@rsmith asked for something slightly different (but I am not sure how to collect that info): He wanted the AST size delta.
For what we have, I think the detailed results table would be more useful if we add some size metric for the files and the increase expressed as a percentage. Perhaps sorting by file size would help contextualize the significance of the percentage change.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D153701/new/
https://reviews.llvm.org/D153701
More information about the cfe-commits
mailing list