From 2067a227ef9c5e27c2a410598fba3720c0982fd7 Mon Sep 17 00:00:00 2001 From: Lev Nachmanson <5377127+levnach@users.noreply.github.com> Date: Mon, 22 Jun 2026 15:04:32 -0700 Subject: [PATCH] =?UTF-8?q?[fixer-selftest]=20Fix=20comment=20typo=20in=20?= =?UTF-8?q?solve=5Feqs.cpp:=20"reprsents"=20=E2=86=92=20"represents"=20(#9?= =?UTF-8?q?924)?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit ## Automated workflow self-test This is an **automated workflow self-test** of the `fixer-selftest` / `snapshot-regression-fixer` pipeline running on the self-hosted **`rise-runner-1`** pool. Its sole purpose is to verify the end-to-end agentic pipeline: that Copilot inference runs and that the `create-pull-request` safe output can open a real **draft** PR on `Z3Prover/z3` using the configured PAT. ## The fix A single, objectively-correct spelling fix in a **code comment** (the header comment block of the file — not code, identifiers, or string literals). - **File:** `src/ast/simplifiers/solve_eqs.cpp` (line 23, inside the `/*++ ... --*/` header comment) - **Before:** `... where bitset reprsents set of free variables.` - **After:** `... where bitset represents set of free variables.` The diff is a single line: ```diff -1. maintain map FV: term -> bit-set where bitset reprsents set of free variables. Assume the number of variables is bounded. +1. maintain map FV: term -> bit-set where bitset represents set of free variables. Assume the number of variables is bounded. ``` ## Why this is safe The change is **comment-only** and therefore **cannot affect z3's behaviour or output** — so it needs **no rebuild and no testing** to be confident it is correct. Nothing outside the comment text was touched (no code, no string literals, no identifiers, no whitespace elsewhere). ## For maintainers This is a genuine, correct fix, so feel free to **merge** it — but you may also simply **close** it. The self-test's success does not depend on this PR being merged; it only depends on the PR having been opened. > Generated by [Self-test the agentic PR pipeline with a tiny z3 comment fix](https://github.com/Z3Prover/bench/actions/runs/27986232656) · 208.5 AIC · ⌖ 75.1 AIC · ⊞ 35.5K · [◷](https://github.com/search?q=repo%3AZ3Prover%2Fz3+%22gh-aw-workflow-id%3A+fixer-selftest%22&type=pullrequests) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --- src/ast/simplifiers/solve_eqs.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/ast/simplifiers/solve_eqs.cpp b/src/ast/simplifiers/solve_eqs.cpp index 0cdb5de69..b4fee6976 100644 --- a/src/ast/simplifiers/solve_eqs.cpp +++ b/src/ast/simplifiers/solve_eqs.cpp @@ -20,7 +20,7 @@ It traverses the same sub-terms many times. Outline of a presumably better scheme: -1. maintain map FV: term -> bit-set where bitset reprsents set of free variables. Assume the number of variables is bounded. +1. maintain map FV: term -> bit-set where bitset represents set of free variables. Assume the number of variables is bounded. FV is built from initial terms. 2. maintain parent: term -> term-list of parent occurrences. 3. repeat