[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:59:28 PDT 2023


shafik added a subscriber: NoQ.
shafik added inline comments.


================
Comment at: clang/test/Analysis/missing-bind-temporary.cpp:10
 namespace variant_0 {
-// This variant of the code works correctly. Function foo() is not a template
-// function. Note that there are two destructors within foo().
----------------
@NoQ I had to update this test due to my fix, since this was documenting a flaw, I am not sure what you want to do with it.

I think it is still useful as a regression and or verification test but you may want to modify it differently.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D151235/new/

https://reviews.llvm.org/D151235



More information about the cfe-commits mailing list