[clang] [clang] Prevent nested RecoveryExpr in BuildConvertedConstantExpression (PR #207072)

[Hrólf Kraki via cfe-commits]

================
@@ -6499,6 +6499,8 @@ static ExprResult BuildConvertedConstantExpression(Sema &S, Expr *From,
     return ExprError();
 
   if (From->containsErrors()) {
+    if (isa<RecoveryExpr>(From))
----------------
Serosh-commits wrote:

> What if the existing RecoveryExpr has a type which is not T? Wasn't that the point of the PR which introduced this check?
> 
> What about not introducing the RecoveryExpr in case the expression already has type T?

addressed it sorry about the delay due to college work are going on i am unable to work on my prs 
really sorry abt that 
thanks for pointing it out :)

> What if the existing RecoveryExpr has a type which is not T? Wasn't that the point of the PR which introduced this check?
> 
> What about not introducing the RecoveryExpr in case the expression already has type T?

sorry for the delay due to college works i  am unable to get time to work on my prs  :\ btw  addressed it 
thanks for pointing out 

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