Merge pull request #15 from cjpatton/faster-pow

fp: Remove no-op multiplications from FieldParameters::pow
divviup · Mar 26, 2021 · 56c194a · 56c194a
2 parents b6dffc6 + cfd5330
commit 56c194a
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/src/fp.rs b/src/fp.rs
@@ -146,7 +146,7 @@ impl FieldParameters {
     /// runtime of this algorithm is linear in the bit length of `exp`.
     pub fn pow(&self, x: u128, exp: u128) -> u128 {
         let mut t = self.elem(1);
-        for i in (0..128).rev() {
+        for i in (0..128 - exp.leading_zeros()).rev() {
             t = self.mul(t, t);
             if (exp >> i) & 1 != 0 {
                 t = self.mul(t, x);