Skip to content

Commit

Permalink
fix: tracy run (#11819)
Browse files Browse the repository at this point in the history
Please read [contributing guidelines](CONTRIBUTING.md) and remove this line.
  • Loading branch information
fcarreiro authored Feb 7, 2025
1 parent f17a8f3 commit fde135d
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -26,6 +26,7 @@ ssh $BOX "
mkdir -p build && cd build && cmake -DCMAKE_MESSAGE_LOG_LEVEL=Warning .. && make -j ;
cd ~/aztec-packages/barretenberg/cpp/ ;
cmake -DCMAKE_MESSAGE_LOG_LEVEL=Warning --preset $PRESET && cmake --build --preset $PRESET --target $BENCHMARK ;
cd ~/tracy/capture/build ;
./tracy-capture -a 127.0.0.1 -f -o trace-$BENCHMARK & ;
sleep 0.1 ;
cd ~/aztec-packages/barretenberg/cpp/build-$PRESET ;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -96,7 +96,7 @@ template <IsUltraFlavor Flavor> void DeciderProvingKey_<Flavor>::allocate_select
template <IsUltraFlavor Flavor>
void DeciderProvingKey_<Flavor>::allocate_table_lookup_polynomials(const Circuit& circuit)
{
PROFILE_THIS_NAME("allocate_table_lookup_polynomials");
PROFILE_THIS_NAME("allocate_table_lookup_polynomials_and_inverses");

size_t table_offset = circuit.blocks.lookup.trace_offset;
// TODO(https://github.com/AztecProtocol/barretenberg/issues/1193): can potentially improve memory footprint
Expand All @@ -113,7 +113,6 @@ void DeciderProvingKey_<Flavor>::allocate_table_lookup_polynomials(const Circuit
// Allocate the read counts and tags polynomials
proving_key.polynomials.lookup_read_counts = Polynomial(max_tables_size, dyadic_circuit_size, table_offset);
proving_key.polynomials.lookup_read_tags = Polynomial(max_tables_size, dyadic_circuit_size, table_offset);
ZoneScopedN("allocating lookup and databus inverses");

const size_t lookup_block_end =
static_cast<size_t>(circuit.blocks.lookup.trace_offset + circuit.blocks.lookup.get_fixed_size(is_structured));
Expand Down

1 comment on commit fde135d

@AztecBot
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Performance Alert ⚠️

Possible performance regression was detected for benchmark 'C++ Benchmark'.
Benchmark result of this commit is worse than the previous benchmark result exceeding threshold 1.05.

Benchmark suite Current: fde135d Previous: ee5fc45 Ratio
wasmClientIVCBench/Full/6 81559.367444 ms/iter 73423.034237 ms/iter 1.11

This comment was automatically generated by workflow using github-action-benchmark.

CC: @ludamad @codygunton

Please sign in to comment.