Skip to content
This repository has been archived by the owner on Jan 26, 2022. It is now read-only.

WIP: ICE v2.3 #77

Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open

WIP: ICE v2.3 #77

wants to merge 9 commits into from

Conversation

PeterTillema
Copy link
Owner

@PeterTillema PeterTillema commented Jul 25, 2018 •
edited
Loading

This is the pull request to use an AST for literally the entire program instead of only expressions.

Fix #85 fix #86 fix #87 fix #88

@PeterTillema PeterTillema self-assigned this Jul 25, 2018
@PeterTillema
Copy link
Owner Author

Progress: I'm quite sure all the operators work properly and (most of) the functions; still need to test them seperately though. This code compiles fine: min(not(A+2/B),B*3+D)/3->A+2*(4/rand->F. I will go on with testing all the inline functions, then other functions like Pause , For( etc, and write tests for them. If that all works, I will try to add floats and matrices.

@PeterTillema PeterTillema changed the title WIP: ICE v3.0 WIP: ICE v2.3 Jul 31, 2018
@PeterTillema
Copy link
Owner Author

Update: I'm postponing ICE v3.0 for now and will focus on getting everything ready with an AST, used to produce smaller code and better optimization. When that is finished, adding floats would be easy-peasy.

@PeterTillema PeterTillema force-pushed the parse-ast branch 3 times, most recently from eb10b00 to 4b1f3cf Compare August 8, 2018 08:52
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees

@PeterTillema PeterTillema

Labels
v3.0
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Better register usage Use AST for entire program NODE optimization framework Optimize comparisons (2)
1 participant
@PeterTillema