Skip to content

Commit

Permalink
fix(avm): execution test ordering (#10226)
Browse files Browse the repository at this point in the history
Please read [contributing guidelines](CONTRIBUTING.md) and remove this
line.
  • Loading branch information
IlyasRidhuan authored Nov 27, 2024
1 parent 11ee884 commit 49b4a6c
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -125,8 +125,8 @@ class AvmExecutionTests : public ::testing::Test {
TEST_F(AvmExecutionTests, basicAddReturn)
{
std::string bytecode_hex = to_hex(OpCode::SET_8) + // opcode SET
"07" // dst_offset 0
"00" // Indirect flag
"07" // dst_offset 0
+ to_hex(AvmMemoryTag::U8) + "00" // val
+ to_hex(OpCode::SET_8) + // opcode SET
"00" // Indirect flag
Expand Down

1 comment on commit 49b4a6c

@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: 49b4a6c Previous: da265b6 Ratio
wasmClientIVCBench/Full/6 89607.03012099999 ms/iter 83771.467434 ms/iter 1.07
wasmconstruct_proof_ultrahonk_power_of_2/20 16946.585014 ms/iter 15206.716956999999 ms/iter 1.11

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

CC: @ludamad @codygunton

Please sign in to comment.