[clang] [Clang] Refactor implementation of "Lifetime extension in range-based for loops" (PR #87930)

https://github.com/yronglin closed https://github.com/llvm/llvm-project/pull/87930 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Clang] Refactor implementation of "Lifetime extension in range-based for loops" (PR #87930)

yronglin wrote: > @yronglin fee free to merge. Thanks! Thanks for your confirmation! https://github.com/llvm/llvm-project/pull/87930 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Clang] Refactor implementation of "Lifetime extension in range-based for loops" (PR #87930)

cor3ntin wrote: @yronglin fee free to merge. Thanks! https://github.com/llvm/llvm-project/pull/87930 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Clang] Refactor implementation of "Lifetime extension in range-based for loops" (PR #87930)

yronglin wrote: friendly ping~ @hubert-reinterpretcast https://github.com/llvm/llvm-project/pull/87930 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Clang] Refactor implementation of "Lifetime extension in range-based for loops" (PR #87930)

yronglin wrote: > Give @hubert-reinterpretcast a few days to possibly review, otherwise LGTM Thanks for your review! https://github.com/llvm/llvm-project/pull/87930 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[clang] [Clang] Refactor implementation of "Lifetime extension in range-based for loops" (PR #87930)

https://github.com/cor3ntin approved this pull request. Give @hubert-reinterpretcast a few days to possibly review, otherwise LGTM https://github.com/llvm/llvm-project/pull/87930 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[clang] [Clang] Refactor implementation of "Lifetime extension in range-based for loops" (PR #87930)

llvmbot wrote: @llvm/pr-subscribers-clang Author: None (yronglin) Changes This PR remove `InMaterializeTemporaryObjectContext` , because it's redundant, materialize non-cv void prvalue temporaries in discarded expressions can only appear under lifetime-extension context. --- Full diff:

[clang] [Clang] Refactor implementation of "Lifetime extension in range-based for loops" (PR #87930)

https://github.com/yronglin created https://github.com/llvm/llvm-project/pull/87930 This PR remove `InMaterializeTemporaryObjectContext` , because it's redundant, materialize non-cv void prvalue temporaries in discarded expressions can only appear under lifetime-extension context. >From