Remove unnecessary FIXME

openvm-org · Jan 17, 2025 · fd19628 · fd19628
1 parent 03af15d
commit fd19628
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/extensions/native/recursion/src/fri/domain.rs b/extensions/native/recursion/src/fri/domain.rs
@@ -116,7 +116,7 @@ where
                 shift: builder.eval(self.shift * domain_power),
                 g,
             };
-            // FIXME: here must use `builder.set_value`. `builder.set` will convert `Usize::Const`
+            // ATTENTION: here must use `builder.set_value`. `builder.set` will convert `Usize::Const`
             // to `Usize::Var` because it calls `builder.eval`.
             builder.set_value(&domains, i, domain);
             builder.assign(&domain_power, domain_power * g_dom);

diff --git a/extensions/native/recursion/src/utils.rs b/extensions/native/recursion/src/utils.rs
@@ -23,7 +23,7 @@ pub fn const_fri_config<C: Config>(
             shift: C::F::ONE,
         };
         let domain_value: TwoAdicMultiplicativeCosetVariable<_> = builder.constant(constant_domain);
-        // FIXME: here must use `builder.set_value`. `builder.set` will convert `Usize::Const`
+        // ATTENTION: here must use `builder.set_value`. `builder.set` will convert `Usize::Const`
         // to `Usize::Var` because it calls `builder.eval`.
         builder.set_value(&subgroups, i, domain_value);
     }