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

Implement codegen for function calls #62

Closed
5 of 6 tasks
hawkw opened this issue Dec 20, 2014 · 7 comments
Closed
5 of 6 tasks

Implement codegen for function calls #62

hawkw opened this issue Dec 20, 2014 · 7 comments
Assignees

Comments

@hawkw
Copy link
Owner

hawkw commented Dec 20, 2014

  • function1.decaf
  • function2.decaf
  • function3.decaf
  • function4.decaf
  • function5.decaf
  • function6.decaf
@hawkw hawkw self-assigned this Dec 20, 2014
@hawkw hawkw added this to the Code Generation milestone Dec 20, 2014
@hawkw
Copy link
Owner Author

hawkw commented Dec 21, 2014

function5.decaf doesn't appear to exist. There's an output file for it in samples, but no source code.

@hawkw
Copy link
Owner Author

hawkw commented Dec 21, 2014

57c4518 implements code gen for calls to local, static functions. function3.decaf works. function1.decaf, function2.decaf, function4.decaf, function6.decaf all are broken because of The Terrible AST Problem (#64).

@hawkw
Copy link
Owner Author

hawkw commented Dec 28, 2014

4910560 fixes function1.decaf.

@hawkw
Copy link
Owner Author

hawkw commented Dec 28, 2014

6daab01 fixes function2.decaf

@hawkw
Copy link
Owner Author

hawkw commented Dec 28, 2014

function6.decaf compiles as of c793e28

@hawkw
Copy link
Owner Author

hawkw commented Dec 28, 2014

The sample source code for function4.decaf actually was wrong, it had a method with the return type int that never returned an integer. I changed the return type to voidin a8f975c and the program now compiles.

@hawkw
Copy link
Owner Author

hawkw commented Dec 28, 2014

Closed by a8f975c.

@hawkw hawkw closed this as completed Dec 28, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant