Skip to content

Conversation

@steakhal
Copy link
Contributor

@steakhal steakhal commented Jan 9, 2025

In my patch there, I left a test expectation stale. Tests with REQUIRES: Z3 never run because no bots check such configurations.

Here I'm adjusting the test expectations to meet reality.

In my patch there, I left a test expectation stale.
Tests with `REQUIRES: Z3` never run because no bots check such
configurations.

Here I'm adjusting the test expectations to meet reality.
@steakhal steakhal added clang Clang issues not falling into any other category clang:static analyzer labels Jan 9, 2025
@steakhal steakhal requested a review from NagyDonat January 9, 2025 13:23
@llvmbot
Copy link
Member

llvmbot commented Jan 9, 2025

@llvm/pr-subscribers-clang

Author: Balazs Benics (steakhal)

Changes

In my patch there, I left a test expectation stale. Tests with REQUIRES: Z3 never run because no bots check such configurations.

Here I'm adjusting the test expectations to meet reality.


Full diff: https://github.com/llvm/llvm-project/pull/122272.diff

1 Files Affected:

  • (modified) clang/test/Analysis/z3/crosscheck-statistics.c (+3-3)
diff --git a/clang/test/Analysis/z3/crosscheck-statistics.c b/clang/test/Analysis/z3/crosscheck-statistics.c
index 7192824c5be31c..8db3df169f246e 100644
--- a/clang/test/Analysis/z3/crosscheck-statistics.c
+++ b/clang/test/Analysis/z3/crosscheck-statistics.c
@@ -28,6 +28,6 @@ int rejecting(int n, int x) {
 // CHECK-NEXT:  1 BugReporter         - Number of reports passed Z3
 // CHECK-NEXT:  1 BugReporter         - Number of reports refuted by Z3
 
-// CHECK:       1 Z3CrosscheckVisitor - Number of Z3 queries accepting a report
-// CHECK-NEXT:  1 Z3CrosscheckVisitor - Number of Z3 queries rejecting a report
-// CHECK-NEXT:  2 Z3CrosscheckVisitor - Number of Z3 queries done
+// CHECK:       1 Z3CrosscheckOracle - Number of Z3 queries accepting a report
+// CHECK-NEXT:  1 Z3CrosscheckOracle - Number of Z3 queries rejecting a report
+// CHECK-NEXT:  2 Z3CrosscheckOracle - Number of Z3 queries done

@llvmbot
Copy link
Member

llvmbot commented Jan 9, 2025

@llvm/pr-subscribers-clang-static-analyzer-1

Author: Balazs Benics (steakhal)

Changes

In my patch there, I left a test expectation stale. Tests with REQUIRES: Z3 never run because no bots check such configurations.

Here I'm adjusting the test expectations to meet reality.


Full diff: https://github.com/llvm/llvm-project/pull/122272.diff

1 Files Affected:

  • (modified) clang/test/Analysis/z3/crosscheck-statistics.c (+3-3)
diff --git a/clang/test/Analysis/z3/crosscheck-statistics.c b/clang/test/Analysis/z3/crosscheck-statistics.c
index 7192824c5be31c..8db3df169f246e 100644
--- a/clang/test/Analysis/z3/crosscheck-statistics.c
+++ b/clang/test/Analysis/z3/crosscheck-statistics.c
@@ -28,6 +28,6 @@ int rejecting(int n, int x) {
 // CHECK-NEXT:  1 BugReporter         - Number of reports passed Z3
 // CHECK-NEXT:  1 BugReporter         - Number of reports refuted by Z3
 
-// CHECK:       1 Z3CrosscheckVisitor - Number of Z3 queries accepting a report
-// CHECK-NEXT:  1 Z3CrosscheckVisitor - Number of Z3 queries rejecting a report
-// CHECK-NEXT:  2 Z3CrosscheckVisitor - Number of Z3 queries done
+// CHECK:       1 Z3CrosscheckOracle - Number of Z3 queries accepting a report
+// CHECK-NEXT:  1 Z3CrosscheckOracle - Number of Z3 queries rejecting a report
+// CHECK-NEXT:  2 Z3CrosscheckOracle - Number of Z3 queries done

@steakhal steakhal merged commit aa0191e into llvm:main Jan 9, 2025
9 of 10 checks passed
@steakhal steakhal deleted the bb/update-stale-z3-crosscheck-test branch January 9, 2025 14:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

clang:static analyzer clang Clang issues not falling into any other category

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants