Skip to content

Commit

Permalink
remove old profile as part of info cmd
Browse files Browse the repository at this point in the history
  • Loading branch information
vezenovm committed Oct 30, 2024
1 parent 321a493 commit 4305f49
Showing 1 changed file with 1 addition and 66 deletions.
67 changes: 1 addition & 66 deletions tooling/nargo_cli/src/cli/info_cmd.rs
Original file line number Diff line number Diff line change
@@ -1,13 +1,10 @@
use std::collections::HashMap;

use acvm::acir::circuit::ExpressionWidth;
use clap::Args;
use iter_extended::vecmap;
use nargo::package::{CrateName, Package};
use nargo_toml::{get_package_manifest, resolve_workspace_from_toml, PackageSelection};
use noirc_artifacts::{debug::DebugArtifact, program::ProgramArtifact};
use noirc_artifacts::program::ProgramArtifact;
use noirc_driver::{CompileOptions, NOIR_ARTIFACT_VERSION_STRING};
use noirc_errors::{debug_info::OpCodesCount, Location};
use prettytable::{row, table, Row};
use rayon::prelude::*;
use serde::Serialize;
Expand Down Expand Up @@ -40,9 +37,6 @@ pub(crate) struct InfoCommand {
#[clap(long, hide = true)]
json: bool,

#[clap(long, hide = true)]
profile_info: bool,

#[clap(flatten)]
compile_options: CompileOptions,
}
Expand Down Expand Up @@ -71,16 +65,6 @@ pub(crate) fn run(args: InfoCommand, config: NargoConfig) -> Result<(), CliError
})
.collect::<Result<_, _>>()?;

if args.profile_info {
for (_, compiled_program) in &binary_packages {
let debug_artifact = DebugArtifact::from(compiled_program.clone());
for function_debug in compiled_program.debug_symbols.debug_infos.iter() {
let span_opcodes = function_debug.count_span_opcodes();
print_span_opcodes(span_opcodes, &debug_artifact);
}
}
}

let program_info = binary_packages
.into_iter()
.par_bridge()
Expand Down Expand Up @@ -114,55 +98,6 @@ pub(crate) fn run(args: InfoCommand, config: NargoConfig) -> Result<(), CliError
Ok(())
}

/// Provides profiling information on
///
/// Number of OpCodes in relation to Noir source file
/// and line number information
fn print_span_opcodes(
span_opcodes_map: HashMap<Location, OpCodesCount>,
debug_artifact: &DebugArtifact,
) {
let mut pairs: Vec<(&Location, &OpCodesCount)> = span_opcodes_map.iter().collect();

pairs.sort_by(|a, b| {
a.1.acir_size.cmp(&b.1.acir_size).then_with(|| a.1.brillig_size.cmp(&b.1.brillig_size))
});

for (location, opcodes_count) in pairs {
let debug_file = debug_artifact.file_map.get(&location.file).unwrap();

let start_byte = byte_index(&debug_file.source, location.span.start() + 1);
let end_byte = byte_index(&debug_file.source, location.span.end() + 1);
let range = start_byte..end_byte;
let span_content = &debug_file.source[range];
let line = debug_artifact.location_line_index(*location).unwrap() + 1;
println!(
"Ln. {}: {} (ACIR:{}, Brillig:{} opcode|s) in file: {}",
line,
span_content,
opcodes_count.acir_size,
opcodes_count.brillig_size,
debug_file.path.to_str().unwrap()
);
}
}
fn byte_index(string: &str, index: u32) -> usize {
let mut byte_index = 0;
let mut char_index = 0;

#[allow(clippy::explicit_counter_loop)]
for (byte_offset, _) in string.char_indices() {
if char_index == index {
return byte_index;
}

byte_index = byte_offset;
char_index += 1;
}

byte_index
}

#[derive(Debug, Default, Serialize)]
struct InfoReport {
programs: Vec<ProgramInfo>,
Expand Down

0 comments on commit 4305f49

Please sign in to comment.