Skip to content

chore(aztec-nr): create a 'with_selector' version of emit_unencrypted_log in avm context#6248

Merged
dbanks12 merged 1 commit intomasterfrom
db/avm-unencrypted-logs-with-selector
May 7, 2024
Merged

chore(aztec-nr): create a 'with_selector' version of emit_unencrypted_log in avm context#6248
dbanks12 merged 1 commit intomasterfrom
db/avm-unencrypted-logs-with-selector

Conversation

@dbanks12
Copy link
Contributor

@dbanks12 dbanks12 commented May 7, 2024

This is necessary since most of the codebase uses a version of this function without a selector arg. It can probably be removed eventually when we settle on an interface.

@dbanks12 dbanks12 requested a review from fcarreiro as a code owner May 7, 2024 16:29
Copy link
Contributor Author

dbanks12 commented May 7, 2024

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @dbanks12 and the rest of your teammates on Graphite Graphite

@github-actions
Copy link
Contributor

github-actions bot commented May 7, 2024

Changes to circuit sizes

Generated at commit: d8e0a5341d4a1f82550cc61f60c143f9297798a2, compared to commit: c571ff0545d54819dd5b386e1bbd932dbe603819

🧾 Summary (100% most significant diffs)

Program ACIR opcodes (+/-) % Circuit size (+/-) %
public_kernel_setup +29 ❌ +0.04% +56 ❌ +0.02%
public_kernel_teardown +29 ❌ +0.04% +56 ❌ +0.02%
public_kernel_app_logic +23 ❌ +0.03% +61 ❌ +0.02%
private_kernel_inner +23 ❌ +0.02% +37 ❌ +0.01%
private_kernel_init +13 ❌ +0.01% +22 ❌ +0.01%
public_kernel_tail +47 ❌ +0.01% +47 ❌ +0.00%
private_kernel_tail -200 ✅ -0.15% +7 ❌ +0.00%
private_kernel_tail_to_public -200 ✅ -0.06% +2 ❌ +0.00%

Full diff report 👇
Program ACIR opcodes (+/-) % Circuit size (+/-) %
public_kernel_setup 71,814 (+29) +0.04% 301,914 (+56) +0.02%
public_kernel_teardown 71,955 (+29) +0.04% 302,050 (+56) +0.02%
public_kernel_app_logic 86,993 (+23) +0.03% 382,839 (+61) +0.02%
private_kernel_inner 106,995 (+23) +0.02% 380,740 (+37) +0.01%
private_kernel_init 91,755 (+13) +0.01% 285,752 (+22) +0.01%
public_kernel_tail 382,097 (+47) +0.01% 1,379,979 (+47) +0.00%
private_kernel_tail 135,680 (-200) -0.15% 1,063,385 (+7) +0.00%
private_kernel_tail_to_public 348,878 (-200) -0.06% 1,536,196 (+2) +0.00%

@dbanks12 dbanks12 changed the title chore(aztec-nr): create a 'with_selector' version of in avm context chore(aztec-nr): create a 'with_selector' version of emit_unencrypted_log in avm context May 7, 2024
@dbanks12 dbanks12 merged commit fda6442 into master May 7, 2024
@dbanks12 dbanks12 deleted the db/avm-unencrypted-logs-with-selector branch May 7, 2024 17:31
TomAFrench added a commit that referenced this pull request May 8, 2024
* master: (25 commits)
  fix: Enable client proof tests (#6249)
  chore: update cspell for abi demonomorphizer (#6258)
  feat(aztec-nr): add 'with_gas()' function to avm call interface (#6256)
  git subrepo push --branch=master noir-projects/aztec-nr
  git_subrepo.sh: Fix parent in .gitrepo file. [skip ci]
  chore: replace relative paths to noir-protocol-circuits
  git subrepo push --branch=master barretenberg
  fix: Pw/update merge check (#6201)
  chore(master): Release 0.37.1 (#6148)
  fix: Cl/split out e2e tests (#6242)
  feat: Typings generator with generics (#6235)
  chore(ci): fix restarts with fresh spot, acir test fixes, non-mandatory benches (#6226)
  chore: misc AVM migration prep changes (#6253)
  feat!: AES blackbox (#6016)
  chore(docs): Fix some typos in specs of private kernel initial (#6224)
  chore(aztec-macros): avm function return types are auto tagged as `pub` (#6250)
  chore(aztec-nr): create a 'with_selector' version of `emit_unencrypted_log` in avm context (#6248)
  fix: registering PublicDataWitness in JsonRpcServer (#6243)
  feat: Sync from noir (#6234)
  feat(avm-simulator): consider previous pending nullifiers across enqueued calls (#6188)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants