Skip to content

fix(avm)!: pre-audit poseidon2 #19963

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
jeanmon wants to merge 26 commits into
base: merge-train/avm
Choose a base branch
from
Open

fix(avm)!: pre-audit poseidon2 #19963

jeanmon wants to merge 26 commits into from
+656 −472

Conversation

@jeanmon
Copy link
Contributor

@jeanmon jeanmon commented Jan 27, 2026

Linear issue: AVM-76

jeanmon added 26 commits January 27, 2026 09:58
@jeanmon
Comments and removing superfluous gating
5b2b264
@jeanmon
Comments in pil related to manually optimized generated file
b044947
@jeanmon
Explanations in pil related to invalid padding value prevention
d77ba51
@jeanmon
Cpp headers cleanup for simulation
7e78442
@jeanmon
Event default initialization
63f1100
@jeanmon
Little code changes/cleanups or comments in simulation code
fcf84ef
@jeanmon
doxygen docs for simulation functions
c05843b
@jeanmon
Cleanup cpp headers in tracegen
79a4ffe
@jeanmon
Using alias C consistently in tracegen
e8654b9
@jeanmon
Tracegen code review for process_hash()
f0da866
@jeanmon
Tracegen review of permutation with memory
e0b3e5c
@jeanmon
Add batch inversion for tracegen of poseidon2_hash
4bc98c0
@jeanmon
Use constant from parameters for the size of the array mapping to the…
47b16c8 
… intermediate columns
@jeanmon
Add doxygen docs for tracegen functions
07e2b77
@jeanmon
Do not implicit cast from bool to FF in tracegen
de4543e
@jeanmon
Use View and CView in optimized relations to be on par with current g…
81e24e6 
…enerated relations
@jeanmon
MD docs small corrections/improvements
29b8a58
@jeanmon
CLaude vulnerability unit tests
076b576
@jeanmon
Comments in poseidon2_mem.pil and tracegen
7c3bdd3
@jeanmon
pil headers and documentation in pil
5f8fbaa
@jeanmon
Some comments and annotations in poseidon2_hash.pil
ebe4e82
@jeanmon
Comments and pointer to new issue
6456344
@jeanmon
Pil comments
fd29b11
@jeanmon
Rename inverse column
b818afe
@jeanmon
Static assertion
da6b979
@jeanmon
Generated files
bd19a32
@jeanmon jeanmon marked this pull request as ready for review January 27, 2026 10:01
@AztecBot
Copy link
Collaborator

AztecBot commented Jan 27, 2026

Flakey Tests

🤖 says: This CI run detected 1 tests that failed, but were tolerated due to a .test_patterns.yml entry.

\033FLAKED\033 (8;;http://ci.aztec-labs.com/5e9dd74160db1a43�5e9dd74160db1a438;;�):  yarn-project/end-to-end/scripts/run_test.sh simple src/e2e_p2p/valid_epoch_pruned_slash.test.ts (401s) (code: 1) group:e2e-p2p-epoch-flakes (\033jeanmon\033: Generated files)

@jeanmon jeanmon requested a review from IlyasRidhuan January 27, 2026 15:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@IlyasRidhuan IlyasRidhuan Awaiting requested review from IlyasRidhuan IlyasRidhuan is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@jeanmon @AztecBot