================
@@ -263,7 +283,11 @@ def plural(num_tests):
         # No tests failed but the build was in a failed state. Bring this to 
the user's
         # attention.
         ninja_failures = find_failure_in_ninja_logs(ninja_logs)
+        all_failures_explained &= are_all_failures_explained(
+            ninja_failures, failure_explanations
+        )
----------------
DavidSpickett wrote:

This could go in the `else` block of `if not ninja_failures`. Then each arm of 
the if has 1 line setting of all_failures_explained.

https://github.com/llvm/llvm-project/pull/172394
_______________________________________________
llvm-branch-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits

Reply via email to