-
Notifications
You must be signed in to change notification settings - Fork 2.2k
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #4055 +/- ##
===========================================
- Coverage 65.72% 65.64% -0.09%
===========================================
Files 307 307
Lines 22733 22733
===========================================
- Hits 14941 14922 -19
- Misses 7792 7811 +19
|
1b42efa
to
d041859
Compare
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.
Ok, but I would like @gcolvin to check it as well.
{ Instruction::SUICIDE, { "SUICIDE", 0, 1, 0, true, Tier::Special } }, | ||
|
||
// these are generated by the interpreter - should never be in user code | ||
{ Instruction::PUSHC, { "PUSHC", 2, 0 , 1, false, Tier::VeryLow } }, | ||
{ Instruction::JUMPC, { "JUMPC", 0, 1, 0, true, Tier::Mid } }, | ||
{ Instruction::JUMPCI, { "JUMPCI", 0, 1, 0, true, Tier::High } }, | ||
{ Instruction::STOP, { "BAD", 0, 0, 0, true, Tier::Zero } }, |
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.
This seems to have been broken all the time. (STOP vs BAD)
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.
Yes, names should match.
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.
Oh - misread, but yes, broken all along.
Fixes #4053.