[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