Skip to content

Splitter fixes#483

Merged
robknight merged 7 commits intomainfrom
splitter_fixes
Feb 20, 2026
Merged

Splitter fixes#483
robknight merged 7 commits intomainfrom
splitter_fixes

Conversation

@robknight
Copy link
Collaborator

@robknight robknight commented Feb 19, 2026

Fixes two issues with the predicate splitter:

  • In cases where public arguments are not used further, drop them from the remaining predicates in the chain
  • When deciding how to weight statements, model the use of private arguments as more expensive than public arguments, to minimise unnecessary predicate-crossing

There are some other minor re-organisations of code, including a fix to an error message and the removal of some unused "wildcard analysis" code.

@robknight robknight requested a review from ed255 February 20, 2026 10:20
Copy link
Collaborator

@ed255 ed255 left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks for fixing this!

@robknight robknight merged commit f6c6ec4 into main Feb 20, 2026
6 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.

2 participants