[clang] [clang][CodeGen] Keep processing the rest of AST after encountering unsupported MC/DC expressions (PR #82464)

[Alan Phipps via cfe-commits]

================
@@ -292,7 +295,7 @@ struct MapRegionCounters : public 
RecursiveASTVisitor<MapRegionCounters> {
                 "contains an operation with a nested boolean expression. "
                 "Expression will not be covered");
             Diag.Report(S->getBeginLoc(), DiagID);
-            return false;
+            return true;
----------------
evodius96 wrote:
Yes, that's what I mean.  There was no benefit in returning false to begin 
with; you have to return true.  The checking that the pass does to catch the 
case, issue the diagnostic, and avoid processing of the data for the expression 
is sufficient to handle the case.

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