[PATCH] D88298: Fix MaterializeTemporaryExpr's type when its an incomplete array.

Erich Keane via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Sep 25 06:12:58 PDT 2020


erichkeane added inline comments.


================
Comment at: clang/lib/Sema/SemaInit.cpp:8061
 
+      QualType MTETy = Step->Type;
+
----------------
It seemed less intrusive/more reliable to fix this during the init-process, and perhaps would fix a few additional cases.  The VarDecl gets updated by the caller of this, and updating both the ExprWtihCleanups and MTE seemed error-prone.


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

https://reviews.llvm.org/D88298



More information about the cfe-commits mailing list