Skip to content

Plumb source locations through classic CLVM evaluation - #556

Draft
prozacchiwawa wants to merge 2 commits into
20260805-classic-codegenfrom
cursor/classic-location-runner-96d3
Draft

prozacchiwawa wants to merge 2 commits into
20260805-classic-codegenfrom
cursor/classic-location-runner-96d3

Conversation

@prozacchiwawa

@prozacchiwawa prozacchiwawa commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

Summary

  • dispatch classic compile-time CLVM evaluation through ClassicAllocator
  • use compiler::clvm::run for SExpClassicAllocator while retaining the existing clvmr-backed TRunProgram path for raw allocators
  • preserve the public TRunProgram interface and use it as the fallback dialect for non-core operators
  • route constant folding, delayed constants, and symbol evaluation through the allocator-selected runner
  • verify that location-aware evaluation preserves distinct result and child spans

Testing

  • cargo test located_allocator_runs_core_clvm_without_losing_result_locations
  • cargo test -- --skip compile_performance_suite

The unfiltered debug test run reached the pre-existing compile_performance_suite after all other library tests passed; that performance test was stopped after more than 22 minutes of sustained CPU usage and was excluded from the completed regression run.

Open in Web Open in Cursor 

cursoragent and others added 2 commits August 10, 2026 21:56
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