Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cpp migration #8

Open
wants to merge 23 commits into
base: dev
Choose a base branch
from
Open

Cpp migration #8

wants to merge 23 commits into from

Conversation

Vipul-Cariappa
Copy link
Owner

@Vipul-Cariappa Vipul-Cariappa commented Feb 4, 2024

TODO

  • update CMakeLists.txt
  • update docs (README.md) for new and updated syntax
  • better error message
    • don't print errors directly to stderr or stdout. redirect to some variable. required for jupyter kernel
    • filename should be displayed
    • semantic failure error message
  • general clean up
  • cli interpreter

  • repetition of same variable name should raise a/an error/warning

Extended Goal. Not in this PR

Looks like I am doing it in this PR itself

  • LLVM support for compiling to native binary.

Following does not yet work
- recursion based on conditional
- global variable declarations
New Compiler: added in some basic function optimizations
everything is working fine, but clean up of code required and safe exits (i.e. remove call to `exit`)
Before Clean Build: 9.141s
Before Successive Build: 7.071s

After Clean Build: 13.112s
Before Successive Build: 3.206s
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