Skip to content

UF fix and various code cleanup - #663

Open
Ovascos wants to merge 13 commits into
SRI-CSL:masterfrom
Ovascos:pr
Open

Ovascos wants to merge 13 commits into
SRI-CSL:masterfrom
Ovascos:pr

Conversation

@Ovascos

@Ovascos Ovascos commented Sep 15, 2026

Copy link
Copy Markdown
Contributor

This PR provides various code clean-ups and performance optimizations:

  • Fixes MCSAT UF Plugin performance bugs #662
  • Clean-up code, improve readability by adding const and joining declaration and definitions of variables.
  • Adds statistics to count backjump-learn and backjump-decide calculus rule application
  • Performance: don't register true_term with mcsat (when a term is solved by the preprocessor)
  • Remove dirty trail replacement from mcsat_analyze_final.
  • Update header file structure to avoid including big headers for one typedef.
  • Adds static to various functions to allow internal linkage

@coveralls

coveralls commented Sep 16, 2026

Copy link
Copy Markdown

Coverage Status

coverage: 71.198% (-0.01%) from 71.209% — Ovascos:pr into SRI-CSL:master

@ahmed-irfan
ahmed-irfan self-requested a review September 17, 2026 07:08
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.

MCSAT UF Plugin performance bugs

2 participants