================
@@ -2177,7 +2177,8 @@ struct CounterCoverageMappingBuilder
   }
 
   void VisitOpaqueValueExpr(const OpaqueValueExpr* OVE) {
-    Visit(OVE->getSourceExpr());
+    if (const Expr *SE = OVE->getSourceExpr())
----------------
bolshakov-a wrote:

Makes sense, thanks! Moreover, I've really found a couple of bugs when some 
expressions referred by `OpaqueValueExpr` are handled more than once (#88881 
and #88898). Regarding `BinaryConditionalOperator`, I've decided to refactor it 
[in a separate PR](https://github.com/llvm/llvm-project/pull/88910) and then to 
introduce the `isUnique()` check in this PR, hoping that there will be no more 
regressions.

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

Reply via email to