b9org / b9

An educational JS virtual machine based on Eclipse OMR
http://www.base9.xyz
Apache License 2.0
45 stars 24 forks source link

Split locals and args #199

Closed nbhuiyan closed 6 years ago

nbhuiyan commented 6 years ago

Signed-off-by: Nazim Uddin Bhuiyan nazimudd@ualberta.ca

nbhuiyan commented 6 years ago

@rwy0717 Ready for another review. There is a merge conflict that I will fix after the changes have been approved.

nbhuiyan commented 6 years ago

@rwy0717 The merge conflicts have been fixed, and the CI build has passed. I think it is ready to be merged.

rwy7 commented 6 years ago

LGTM. Now that we differentiate between locals and arguments, we should take another look at how the JIT feature flags (passparam, lazy-vm-state) affect how locals and parameters behave. Thanks @nbhuiyan. closes #154.