-
-
Notifications
You must be signed in to change notification settings - Fork 5
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 tests on BitwiseOp::Shl #13
Conversation
Codecov Report
@@ Coverage Diff @@
## master #13 +/- ##
========================================
+ Coverage 15.6% 16.2% +0.6%
========================================
Files 10 10
Lines 2271 2271
========================================
+ Hits 354 368 +14
+ Misses 1917 1903 -14 see 2 files with indirect coverage changes 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
tests/asm.rs
Outdated
eq.n r8192[0],r8192[2]; | ||
ret; | ||
} | ||
// FIXME shift value is limited to 1024 |
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.
aff0e33
to
bf3b1d0
Compare
src/pipelines/compiler.rs
Outdated
@@ -505,6 +505,7 @@ impl<'i> Statement<'i> { | |||
} | |||
|
|||
fn goto(&'i self, no: u8, issues: &mut Issues<'i, issues::Compile>) -> Option<String> { | |||
println!("aaaaaaaaaaaa {:?}", self.operands.get(0)); |
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.
I think we need to remove this
src/pipelines/compiler.rs
Outdated
@@ -529,7 +530,7 @@ impl<'i> Statement<'i> { | |||
SemanticError::OperandMissed { | |||
operator: self.operator.0, | |||
pos: no + 1, | |||
expected: "goto statement", | |||
expected: "goto statemelllnt", |
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.
Looks like a typo / accidental 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.
thank you
Why do the tests are failing? |
dependent on AluVM/rust-aluvm#91 |
With AluVM/rust-aluvm#91 merged we can rebase now and then I will merge this |
rebased |
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.
ACK 51fcc8a
No description provided.