[AST][RecoveryExpr] Fix the missing type when rebuilding RecoveryExpr in TreeTransform.
authorHaojian Wu <hokein.wu@gmail.com>
Fri, 7 Aug 2020 09:14:37 +0000 (11:14 +0200)
committerHaojian Wu <hokein.wu@gmail.com>
Fri, 7 Aug 2020 09:15:19 +0000 (11:15 +0200)
Differential Revision: https://reviews.llvm.org/D85423

clang/lib/Sema/TreeTransform.h
clang/test/SemaTemplate/recovery-tree-transform-crash.cpp [new file with mode: 0644]

index 29d263d..d23b441 100644 (file)
@@ -3618,8 +3618,8 @@ public:
   }
 
   ExprResult RebuildRecoveryExpr(SourceLocation BeginLoc, SourceLocation EndLoc,
-                                 ArrayRef<Expr *> SubExprs) {
-    return getSema().CreateRecoveryExpr(BeginLoc, EndLoc, SubExprs);
+                                 ArrayRef<Expr *> SubExprs, QualType Type) {
+    return getSema().CreateRecoveryExpr(BeginLoc, EndLoc, SubExprs, Type);
   }
 
 private:
@@ -10209,7 +10209,7 @@ ExprResult TreeTransform<Derived>::TransformRecoveryExpr(RecoveryExpr *E) {
   if (!getDerived().AlwaysRebuild() && !Changed)
     return E;
   return getDerived().RebuildRecoveryExpr(E->getBeginLoc(), E->getEndLoc(),
-                                          Children);
+                                          Children, E->getType());
 }
 
 template<typename Derived>
diff --git a/clang/test/SemaTemplate/recovery-tree-transform-crash.cpp b/clang/test/SemaTemplate/recovery-tree-transform-crash.cpp
new file mode 100644 (file)
index 0000000..9776aeb
--- /dev/null
@@ -0,0 +1,14 @@
+// RUN: %clang_cc1 -verify -frecovery-ast -frecovery-ast-type %s
+
+template <class T> struct Ptr { T *operator->() const; };
+
+struct ABC {
+  void run();
+};
+
+Ptr<ABC> call(int); // expected-note {{candidate function not viable}}
+
+void test() {
+  call()->run(undef); // expected-error {{no matching function for call to 'call'}} \
+                         expected-error {{use of undeclared identifier}}
+}