-
-
Notifications
You must be signed in to change notification settings - Fork 411
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
[Merged by Bors] - Generate Opcode
impl using macro
#2391
Conversation
Test262 conformance changes
|
Codecov Report
@@ Coverage Diff @@
## main #2391 +/- ##
==========================================
- Coverage 39.81% 39.59% -0.22%
==========================================
Files 307 307
Lines 23547 23394 -153
==========================================
- Hits 9375 9264 -111
+ Misses 14172 14130 -42
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
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.
Great change!
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.
Great change! Hopefully it won't add much compile time :)
997cb6b
to
f5d090a
Compare
bors r+ |
Pull request successfully merged into main. Build succeeded: |
Opcode
impl using macroOpcode
impl using macro
This reduces a lot of the repetition from the
Opcode
functions and makes it easier to add a new Op.