Skip to content

Commit

Permalink
change jit logs to debug for now (#1957)
Browse files Browse the repository at this point in the history
  • Loading branch information
leonardoalt authored Oct 25, 2024
1 parent c40f28e commit 28a8022
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 3 deletions.
3 changes: 2 additions & 1 deletion executor/src/constant_evaluator/jit_compiler.rs
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,8 @@ pub fn generate_values<T: FieldElement>(
) -> HashMap<(String, PolyID), VariablySizedColumn<T>> {
let compiled_pil = match powdr_jit_compiler::compile(analyzed, &symbols_to_compile(analyzed)) {
Err(err) => {
log::error!("Failed to compile some constant columns: {}", err);
// TODO this should be changed back to Error after the introduction of the ToCol trait.
log::debug!("Failed to compile some constant columns: {}", err);
return HashMap::new();
}
Ok(fun_map) => fun_map,
Expand Down
6 changes: 4 additions & 2 deletions jit-compiler/src/lib.rs
Original file line number Diff line number Diff line change
Expand Up @@ -53,7 +53,8 @@ pub fn compile<T: FieldElement>(
analyzed: &Analyzed<T>,
requested_symbols: &[&str],
) -> Result<CompiledPIL, String> {
log::info!("JIT-compiling {} symbols...", requested_symbols.len());
// TODO this should be changed back to Info after the introduction of the ToCol trait.
log::debug!("JIT-compiling {} symbols...", requested_symbols.len());

let mut codegen = CodeGenerator::new(analyzed);
let successful_symbols = requested_symbols
Expand All @@ -70,7 +71,8 @@ pub fn compile<T: FieldElement>(

if successful_symbols.len() < requested_symbols.len() {
let successful_hash = successful_symbol_names.iter().collect::<HashSet<_>>();
log::info!(
// TODO this should be changed back to Info after the introduction of the ToCol trait.
log::debug!(
"Unable to generate code during JIT-compilation for the following symbols. Will use evaluator instead.\n{}",
requested_symbols
.iter()
Expand Down

0 comments on commit 28a8022

Please sign in to comment.