Skip to content

Avoid llvm-kompile when running proofs - rebooted#1079

Merged
dkcumming merged 4 commits into
masterfrom
jb/no-llvm-kompile-rebooted
Apr 27, 2026
Merged

Avoid llvm-kompile when running proofs - rebooted#1079
dkcumming merged 4 commits into
masterfrom
jb/no-llvm-kompile-rebooted

Conversation

@jberthold
Copy link
Copy Markdown
Collaborator

@jberthold jberthold commented Apr 27, 2026

  • Do not extend and recompile the definition.kore file for LLVM, only extend the Haskell one
  • Mark the lookupTy, lookupAlloc and lookupFunction as no-evaluators (including the stratified helper functions)
  • Also mark (transitively) all functions whose equations contain these lookup functions or other no-evaluators functions on the RHS as no-evaluators (the LLVM backend will core-dump when it encounters a no-evaluators function).

Integration tests run considerably faster.

  • Integration tests
  • Test this with Solana token proofs

* Lookup functions have `no-evaluators` attributes and no defaults
  - LLVM backend won't be called to evaluate expressions containing them
  - HS backend (booster) will evaluate
* No LLVM `kompile` step before executing symbolically
* Only the HS definition.kore is extended with the lookup function equations
@dkcumming dkcumming self-assigned this Apr 27, 2026
Copy link
Copy Markdown
Collaborator

@dkcumming dkcumming left a comment

Choose a reason for hiding this comment

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

Awesome stuff. Mir semantics test suite passes in 30% of time previously. Solana P-Token proofs also appear to have slight improvement.

@dkcumming dkcumming merged commit d5a5df9 into master Apr 27, 2026
12 checks passed
@dkcumming dkcumming deleted the jb/no-llvm-kompile-rebooted branch April 27, 2026 21:51
@Stevengre
Copy link
Copy Markdown
Contributor

Amazing!

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