Skip to content
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

Implement u32 shl, shr, eq, neq, not #58

Merged
merged 1 commit into from
Jan 18, 2022
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
135 changes: 119 additions & 16 deletions assembly/src/parsers/u32_ops.rs
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
use super::{AssemblyError, Operation, Token};
use super::{parse_int_param, AssemblyError, BaseElement, Operation, Token};

// CONVERSIONS AND TESTS
// ================================================================================================
Expand All @@ -8,7 +8,10 @@ pub fn parse_u32test(span_ops: &mut Vec<Operation>, op: &Token) -> Result<(), As
match op.num_parts() {
0 => return Err(AssemblyError::missing_param(op)),
1 => {
span_ops.push(Operation::Dup0);
span_ops.push(Operation::U32split);
span_ops.push(Operation::Swap);
span_ops.push(Operation::Drop);
span_ops.push(Operation::Eqz);
}
_ => return Err(AssemblyError::extra_param(op)),
Expand Down Expand Up @@ -186,19 +189,74 @@ pub fn parse_u32xor(span_ops: &mut Vec<Operation>, op: &Token) -> Result<(), Ass
Ok(())
}

/// TODO: implement
pub fn parse_u32not(_span_ops: &mut Vec<Operation>, _op: &Token) -> Result<(), AssemblyError> {
unimplemented!()
/// Translates u32not assembly instruction to VM operation PUSH(2^32) SWAP INCR U32SUB.
/// The reason this works is because 2^32 provides a bit mask of ones, which after
/// subtracting the element, flips the bits of the original value to perform a bitwise NOT.
pub fn parse_u32not(span_ops: &mut Vec<Operation>, op: &Token) -> Result<(), AssemblyError> {
match op.num_parts() {
0 => return Err(AssemblyError::missing_param(op)),
1 => {
// Assert the value is u32
span_ops.push(Operation::U32split);
span_ops.push(Operation::Not);
span_ops.push(Operation::Assert);

span_ops.push(Operation::Push(BaseElement::new(2u64.pow(32))));
span_ops.push(Operation::Swap);
span_ops.push(Operation::Incr);
span_ops.push(Operation::U32sub);

// Drop the underflow flag
span_ops.push(Operation::Drop);
}
_ => return Err(AssemblyError::extra_param(op)),
}

Ok(())
}

/// TODO: implement
pub fn parse_u32shl(_span_ops: &mut Vec<Operation>, _op: &Token) -> Result<(), AssemblyError> {
unimplemented!()
/// Translates u32shl.x assembly instruction to VM operation PUSH(2^x) MUL U32SPLIT DROP.
pub fn parse_u32shl(span_ops: &mut Vec<Operation>, op: &Token) -> Result<(), AssemblyError> {
match op.num_parts() {
0..=1 => return Err(AssemblyError::missing_param(op)),
2 => {
// Assert the value is u32
span_ops.push(Operation::U32split);
span_ops.push(Operation::Not);
span_ops.push(Operation::Assert);

let x = parse_int_param(op, 1, 1, 31)?;
span_ops.push(Operation::Push(BaseElement::new(2u64.pow(x))));
span_ops.push(Operation::Mul);
span_ops.push(Operation::U32split);
span_ops.push(Operation::Drop);
}
_ => return Err(AssemblyError::extra_param(op)),
}

Ok(())
}

/// TODO: implement
pub fn parse_u32shr(_span_ops: &mut Vec<Operation>, _op: &Token) -> Result<(), AssemblyError> {
unimplemented!()
/// Translates u32shl.x assembly instruction to VM operation PUSH(2^x) U32DIV SWAP DROP.
pub fn parse_u32shr(span_ops: &mut Vec<Operation>, op: &Token) -> Result<(), AssemblyError> {
match op.num_parts() {
0..=1 => return Err(AssemblyError::missing_param(op)),
2 => {
// Assert the value is u32
span_ops.push(Operation::U32split);
span_ops.push(Operation::Not);
span_ops.push(Operation::Assert);

let x = parse_int_param(op, 1, 1, 31)?;
span_ops.push(Operation::Push(BaseElement::new(2u64.pow(x))));
span_ops.push(Operation::U32div);
span_ops.push(Operation::Swap);
span_ops.push(Operation::Drop);
}
_ => return Err(AssemblyError::extra_param(op)),
}

Ok(())
}

/// TODO: implement
Expand All @@ -219,14 +277,59 @@ pub fn parse_u32revb(_span_ops: &mut Vec<Operation>, _op: &Token) -> Result<(),
// COMPARISON OPERATIONS
// ================================================================================================

/// TODO: implement
pub fn parse_u32eq(_span_ops: &mut Vec<Operation>, _op: &Token) -> Result<(), AssemblyError> {
unimplemented!()
/// Translates u32eq assembly instruction to VM operations.
/// Specifically we test the first two numbers to be u32 (U32SPLIT NOT ASSERT),
/// then perform a EQ to check the equality.
pub fn parse_u32eq(span_ops: &mut Vec<Operation>, op: &Token) -> Result<(), AssemblyError> {
match op.num_parts() {
0 => return Err(AssemblyError::missing_param(op)),
1 => {
// Check first number is u32
span_ops.push(Operation::U32split);
span_ops.push(Operation::Not);
span_ops.push(Operation::Assert);

span_ops.push(Operation::Swap);

// Check second number is u32
span_ops.push(Operation::U32split);
span_ops.push(Operation::Not);
span_ops.push(Operation::Assert);

span_ops.push(Operation::Eq);
}
_ => return Err(AssemblyError::extra_param(op)),
}

Ok(())
}

/// TODO: implement
pub fn parse_u32neq(_span_ops: &mut Vec<Operation>, _op: &Token) -> Result<(), AssemblyError> {
unimplemented!()
/// Translates u32neq assembly instruction to VM operations.
/// Specifically we test the first two numbers to be u32 (U32SPLIT NOT ASSERT),
/// then perform a EQ NOT to check the equality.
pub fn parse_u32neq(span_ops: &mut Vec<Operation>, op: &Token) -> Result<(), AssemblyError> {
match op.num_parts() {
0 => return Err(AssemblyError::missing_param(op)),
1 => {
// Check first number is u32
span_ops.push(Operation::U32split);
span_ops.push(Operation::Not);
span_ops.push(Operation::Assert);

span_ops.push(Operation::Swap);

// Check second number is u32
span_ops.push(Operation::U32split);
span_ops.push(Operation::Not);
span_ops.push(Operation::Assert);

span_ops.push(Operation::Eq);
span_ops.push(Operation::Not);
}
_ => return Err(AssemblyError::extra_param(op)),
}

Ok(())
}

/// TODO: implement
Expand Down
2 changes: 1 addition & 1 deletion core/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,6 @@ default = ["std"]
std = ["math/std", "winter-utils/std"]

[dependencies]
crypto = { package = "winter-crypto", version = "0.3", default-features = false }
crypto = { package = "winter-crypto", version = "0.3.1", default-features = false }
math = { package = "winter-math", version = "0.3", default-features = false }
winter-utils = { package = "winter-utils", version = "0.3", default-features = false }