-
Notifications
You must be signed in to change notification settings - Fork 824
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
Use Undefined instruction with payload to carry over TrapInformation on Singlepass #2836
Conversation
…without using an vector or hashmap on Singlepass
Tests seem to be failing |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You can probably also remove TrapCode::StackOverflow
and TrapCode::HeapAccessOutOfBounds
from singlepass since those trap codes are now detected in the signal handler.
… Singlepass (to fix Trap opcode tracking)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Apart from that, LGTM.
Based on the comment from Amanieu, we can merge manually (tests are already passing for everything) |
bors r+ |
2836: Use Undefined instruction with payload to carry over TrapInformation on Singlepass r=ptitSeb a=ptitSeb # Description Use Undefined instruction with payload to carry over TrapInformation on without using an vector or hashmap on Singlepass. This allow for slightly more compact binary and slightly faster trap decoding on runtime. Co-authored-by: ptitSeb <[email protected]>
Timed out. |
bors retry |
Already running a review |
Those TimeOut seems to be more and more frequent now. There is an issue with the CI. |
Description
Use Undefined instruction with payload to carry over TrapInformation on without using an vector or hashmap on Singlepass.
This allow for slightly more compact binary and slightly faster trap decoding on runtime.