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

Wentao Zhang via cfe-commits cfe-commits at lists.llvm.org
Thu Feb 22 10:51:43 PST 2024


================
@@ -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


More information about the cfe-commits mailing list