Skip to content

Remove z3 constraint solver option - #5055

Merged
bruntib merged 3 commits into
Ericsson:masterfrom
dkrupp:remove-z3-constraint-solver-option
Aug 31, 2026
Merged

Remove z3 constraint solver option#5055
bruntib merged 3 commits into
Ericsson:masterfrom
dkrupp:remove-z3-constraint-solver-option

Conversation

@dkrupp

@dkrupp dkrupp commented Aug 28, 2026

Copy link
Copy Markdown
Member

Backward Incompatible: Remove --z3 flag

Since the z3 solver it is officially unsupported in clangsa
and it is inherently slow for production use, the flag is removed.

Z3 refutation for clang static analyzer will continue to be supported!

gamesh411 and others added 2 commits August 5, 2026 10:29
Upstream Clang removed -analyzer-constraints=z3, replacing it with
unsupported-z3 (llvm/llvm-project#205370). CodeChecker no longer
injects this flag into the analyzer command line.

- --z3 flag is kept but deprecated (no-op with warning), removal in 6.30
- is_z3_capable() probes both unsupported-z3 and z3 for older clangs
- Z3 refutation (--z3-refutation) is unaffected
Since the z3 solver it is officially unsupported in clangsa
and it is inherently slow for production use, the flag is removed.
@dkrupp
dkrupp requested a review from bruntib as a code owner August 28, 2026 13:24
@dkrupp dkrupp added this to the release 6.29.0 milestone Aug 28, 2026
@dkrupp

dkrupp commented Aug 28, 2026

Copy link
Copy Markdown
Member Author

This PR is based on #5011 from @gamesh411

Comment on lines +745 to +751
handler.enable_z3 = 'enable_z3' in args and args.enable_z3 == 'on'
if handler.enable_z3:
LOG.warning("The '--z3' flag is deprecated and has no effect. "
"The Z3 constraint solver backend is no longer "
"supported by upstream Clang. This flag will be "
"removed in CodeChecker 6.30. Use "
"'--z3-refutation' instead.")

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The --z3 flag has been eliminated, so this can never be True. This whole section could be removed.

Comment thread docs/analyzer/checker_and_analyzer_configuration.md Outdated
Comment thread docs/analyzer/user_guide.md Outdated
Comment thread docs/analyzer/user_guide.md Outdated
@dkrupp
dkrupp requested a review from bruntib August 31, 2026 13:07
@bruntib
bruntib merged commit f4d05a3 into Ericsson:master Aug 31, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants