Explain why the error we produce isn't very informative

noir-lang · Dec 18, 2024 · 989cb6a · 989cb6a
1 parent 2d6c5a0
commit 989cb6a
Showing 1 changed file with 2 additions and 0 deletions.
diff --git a/compiler/noirc_frontend/src/hir/comptime/interpreter.rs b/compiler/noirc_frontend/src/hir/comptime/interpreter.rs
@@ -575,6 +575,8 @@ impl<'local, 'interner> Interpreter<'local, 'interner> {
                 match &global_info.value {
                     GlobalValue::Resolved(value) => Ok(value.clone()),
                     GlobalValue::Resolving => {
+                        // Note that the error we issue here isn't very informative (it doesn't include the actual cycle)
+                        // but the general dependency cycle detector will give a better error later on during compilation.
                         let location = self.elaborator.interner.expr_location(&id);
                         Err(InterpreterError::GlobalsDependencyCycle { location })
                     }