Skip to content

Add deinline timing diagnostics#467

Draft
prozacchiwawa wants to merge 6 commits intomainfrom
cursor/deinline-diagnostics-2432
Draft

Add deinline timing diagnostics#467
prozacchiwawa wants to merge 6 commits intomainfrom
cursor/deinline-diagnostics-2432

Conversation

@prozacchiwawa
Copy link
Copy Markdown
Contributor

@prozacchiwawa prozacchiwawa commented May 6, 2026

Summary

  • add opt-in deinline timing diagnostics written as TSV output in the current working directory
  • track function cache hits and misses to report cache hit percentage per deinline pass
  • keep skipped/no-helper rows available via an explicit diagnostics env var while keeping workflow output focused on active deinline work
  • enable diagnostics in the Chia Gaming workflow and print a compact slow-pass summary
  • persist function-cache entries under .chialisp/function-cache/ using a sha256tree hash of the function cache key in each filename
  • include compile settings in function cache keys so persisted entries do not cross dialect/module-phase boundaries
  • update function-cache tests for disk persistence and explicit setup for the new counters

Testing

  • cargo fmt --all
  • cargo test codegen_funcache -- --nocapture
  • cargo test diskcache -- --nocapture
  • cargo clippy --workspace -- -D warnings
  • CHIALISP_DEINLINE_DIAGNOSTICS=deinline-two-program.tsv CHIALISP_DEINLINE_DIAGNOSTICS_SKIPPED=1 cargo test test_two_program_import_include -- --nocapture
  • Targeted Chia Gaming calpoker rebuild benchmark, after prebuilding Rust with CHIALISP_NOCOMPILE=1 and limiting chialisp.toml to calpoker-generate and calpoker-generate-v1:
    • cold rebuild with empty .chialisp: real 2002.01s, diagnostics aggregate hit rate 51.63%
    • warm rebuild preserving only .chialisp/function-cache: real 1892.07s, diagnostics aggregate hit rate 81.62%
    • warm rebuild preserving all .chialisp cache data: real 47.60s
Open in Web Open in Cursor 

cursoragent and others added 6 commits May 6, 2026 06:25
Co-authored-by: arty <prozacchiwawa@users.noreply.github.com>
Co-authored-by: arty <prozacchiwawa@users.noreply.github.com>
Co-authored-by: arty <prozacchiwawa@users.noreply.github.com>
Co-authored-by: arty <prozacchiwawa@users.noreply.github.com>
Co-authored-by: arty <prozacchiwawa@users.noreply.github.com>
Co-authored-by: arty <prozacchiwawa@users.noreply.github.com>
Base automatically changed from 20260217-introspective-constants to main May 7, 2026 22:12
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