Skip to content

Maybe gc enabled fdsklsl#2029

Merged
andrew4328 merged 34 commits intoLambda-Mountain-Compiler-Backend:mainfrom
andrew4328:maybe-gc-enabled-fdsklsl
Apr 9, 2026
Merged

Maybe gc enabled fdsklsl#2029
andrew4328 merged 34 commits intoLambda-Mountain-Compiler-Backend:mainfrom
andrew4328:maybe-gc-enabled-fdsklsl

Conversation

@andrew4328
Copy link
Copy Markdown
Collaborator

@andrew4328 andrew4328 commented Apr 9, 2026

Describe your changes

  • new Type case TId for numerical ids
  • change Phi::Ids to use TIds instead of string identifiers
  • integers don't use any heap memory
  • integers are very fast to compare because they never need to load indirect memory like strcmp
  • about 10-20x speedup and significantly reduced memory usage for GC enabled compiler
  • full test suite takes 9.5 minutes to run with GC enabled for all tests

Issue ticket number and link

#1895

Checklist before requesting a review

  • [ x ] I have performed an AI-assisted self-review of my code.
Can you review my pull request and provide some suggestions?
https://patch-diff.githubusercontent.com/raw/Lambda-Mountain-Compiler-Backend/lambda-mountain/pull/1926.diff
  • [ x ] If it is a new feature, I have added thorough tests.
  • [ x ] I agree to release these changes under the terms of the permissive MIT license (1).
  1. https://github.com/andrew-johnson-4/lambda-mountain/blob/main/LICENSE

@andrew4328 andrew4328 merged commit cc776c7 into Lambda-Mountain-Compiler-Backend:main Apr 9, 2026
1 check passed
@andrew4328 andrew4328 deleted the maybe-gc-enabled-fdsklsl branch April 9, 2026 20:42
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.

1 participant