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

[Wentao Zhang 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;
----------------
whentojump wrote:

Actually my point is: returning `false` would exit early from the AST 
traversal, and I suppose both places *have to* return `true`. Otherwise we'd 
observe things in my original post.




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