Skip to content

Add runner type to ClassicAllocator - #560

Draft
prozacchiwawa wants to merge 2 commits into
20260805-classic-codegenfrom
cursor/classic-allocator-runner-type-3089
Draft

prozacchiwawa wants to merge 2 commits into
20260805-classic-codegenfrom
cursor/classic-allocator-runner-type-3089

Conversation

@prozacchiwawa

@prozacchiwawa prozacchiwawa commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

Summary

  • add an associated runner type to ClassicAllocator, constrained to clone and dereference to TRunProgram
  • use Rc<dyn TRunProgram> for both existing allocator implementations
  • update allocator-generic classic compiler functions to accept the associated runner type
  • make affected test calls explicit where associated-type inference prevents runner coercion

Testing

  • cargo check --all-targets
  • cargo test --lib tests::classic::stage_2 (17 passed)
  • cargo test --lib -- --skip compile_performance_suite (705 passed, 1 ignored, 1 filtered)
  • rustfmt --edition 2021 --check on all changed Rust files

The unfiltered library suite reached only the existing long-running compile_performance_suite; it was stopped after the other tests completed and then rerun successfully with that performance test excluded.

Open in Web Open in Cursor 

cursoragent and others added 2 commits August 12, 2026 20:58
Co-authored-by: arty <prozacchiwawa@users.noreply.github.com>
Co-authored-by: arty <prozacchiwawa@users.noreply.github.com>
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