[llvm-branch-commits] [clang] [clang] simplify placeholder type deduction for constant template parameters (PR #160439)

[Erich Keane via llvm-branch-commits]

================
@@ -8595,6 +8588,11 @@ static bool CheckNonTypeTemplatePartialSpecializationArgs(
       continue;
     }
 
+    if (isa<RecoveryExpr>(ArgExpr)) {
+      HasError = true;
----------------
erichkeane wrote:

This doesn't seem quite right.  We care not only if this is a `RecoveryExpr`, but if ANYTHING inside of it does, right?  I think what you really want is for this to be `ArgExpr->containsErrors()`.

https://github.com/llvm/llvm-project/pull/160439