================
@@ -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
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to