[PATCH] D151235: [Clang] Switch from TransformExpr to TransformInitializer in places we need to revert initializer to it syntactic form for Sema
Shafik Yaghmour via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue May 23 10:56:18 PDT 2023
shafik created this revision.
shafik added reviewers: rsmith, aaron.ballman, erichkeane.
Herald added a project: All.
shafik requested review of this revision.
In some cases we are using `TransformExpr` instead of `TransformInitializer`, this results in `ExprWithCleanups` being dropped and we are not emitting a destructor as a result.
This fixes: https://github.com/llvm/llvm-project/issues/62818
https://reviews.llvm.org/D151235
Files:
clang/lib/Sema/TreeTransform.h
clang/test/Analysis/missing-bind-temporary.cpp
clang/test/CodeGenCXX/gh62818.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D151235.524799.patch
Type: text/x-patch
Size: 4175 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20230523/d42c29a8/attachment.bin>
More information about the cfe-commits
mailing list