Move DATA* opcodes to the D section #46
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Closes #43
There is a conflict of EXCHANGE and DATALOAD and the E section is full.
It has been noted the E section could be for Emmediates, but we'd need to separate a bunch of opcodes otherwise related, and would likely run out of opcode slots in E section anyway quite soon.
My quick pitch for the move to D is to instead leave opcodes logically bound to EOF in
E
leaving plenty of space there for the future. As such,D
ataand stacklands in D.Let's quickly gather feedback on this. If there's objections, I'll do an alternative PR to do the Emmediates change and we go with that instead.