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
Add: allow use of switches and random switches as procedures #66
Conversation
4430190
to
4828210
Compare
I've tested this with a single stored procedure in Iron Horse. This replaces global-switch-and-registers with checking a procedure result. It makes more readable code. It's not a very interesting test case but eh. Appears to work. |
I think it's ready now. |
One final +1: I have converted Iron Horse to use procedures widely, and have compiled hundreds of times now using nmlc built with this PR. Zero issues so far. |
NFO output seems correct, but ID handling and generation needs work. For now 2 different procedures can have the same ID.
Work probably needed in other areas too. That's why it's a draft.
Feel free to add commits in my branch.