================
@@ -34,21 +34,22 @@ void test_invalid_call_1(int s) {
 
 int some_func2(int a, int b);
 void test_invalid_call_2() {
-  // CHECK:   -RecoveryExpr {{.*}} 'int' contains-errors
+  // CHECK:   -RecoveryExpr {{.*}} '<dependent type>' contains-errors
----------------
zyn0217 wrote:

We used to call ActOnExpr for this case, which ended up creating a 
non-dependent RecoveryExpr with its type heuristically computed over the 
overload candidate set, in `FinishOverloadedCallExpr`.

We could do the same again by calling `ActOnCallExpr` anyway, but that would 
lead to a lot more diagnostic regressions when recovering from an 
already-diagnosed function-call-like expression.

Or do you prefer assigning Context.IntTy for these RecoveryExpr expressions, 
regardless?

https://github.com/llvm/llvm-project/pull/136295
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to