Skip to content

20260805 classic codegen - #553

Open
prozacchiwawa wants to merge 40 commits into
mainfrom
20260805-classic-codegen
Open

20260805 classic codegen#553
prozacchiwawa wants to merge 40 commits into
mainfrom
20260805-classic-codegen

Conversation

@prozacchiwawa

@prozacchiwawa prozacchiwawa commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

Note

High Risk
Large refactor of the classic compilation pipeline with new language features (&rest, @env, modern com assembly) that can change generated CLVM and error reporting across many code paths.

Overview
Introduces a ClassicAllocator abstraction (with ClError carrying Srcloc) and SExpClassicAllocator, which keeps modern compiler S-expressions and raw CLVM nodes in sync so stage-2 can report source locations instead of only NodePtr-based EvalErr.

Stage-2 com, mod, pattern matching, and shared sexp helpers are generalized over that trait; compile_application gains &rest tail handling, and defun-inline can reference @*env* via new inline-environment lowering.

For strict/modern dialects, com now runs assemble_com_program to turn symbolic operator heads (e.g. q) into numeric opcodes while preserving per-subexpression locations; @*env* is also recognized like @ during symbol transformation.

CI updates mandelbrot-cldb.py with optional sigil substitution and YAML-based output comparison (plus pyyaml in the workflow).

Reviewed by Cursor Bugbot for commit 5299534. Bugbot is set up for automated code reviews on this repo. Configure here.

prozacchiwawa and others added 30 commits July 23, 2026 12:38
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>
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>
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>
prozacchiwawa and others added 9 commits August 4, 2026 15:40
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>

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes using high effort and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Want fixes drafted automatically? Bugbot Autofix can create code changes for findings. A team admin can enable Autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 299c335. Configure here.

Comment thread .github/workflows/build-test.yml Outdated
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