fix: reimplement monty optimizations

pluto · May 6, 2024 · 034ef34 · 034ef34
1 parent 6629016
commit 034ef34
Showing 1 changed file with 2 additions and 4 deletions.
diff --git a/src/field/gf_101.rs b/src/field/gf_101.rs
@@ -33,8 +33,7 @@ impl fmt::Display for GF101 {
 }
 
 impl GF101 {
-  // pub const fn new(value: u32) -> Self { Self { value: to_monty(value) } }
-  pub const fn new(value: u32) -> Self { Self { value: value % Self::ORDER } }
+  pub const fn new(value: u32) -> Self { Self { value: to_monty(value) } }
 }
 
 impl FiniteField for GF101 {
@@ -121,8 +120,7 @@ impl SubAssign for GF101 {
 impl Mul for GF101 {
   type Output = Self;
 
-  // fn mul(self, rhs: Self) -> Self { Self { value: from_monty(self.value * rhs.value) } }
-  fn mul(self, rhs: Self) -> Self { Self { value: (self.value * rhs.value) % Self::ORDER } }
+  fn mul(self, rhs: Self) -> Self { Self { value: from_monty(self.value * rhs.value) } }
 }
 
 impl MulAssign for GF101 {