Skip to content

Conversation

@akokoshn
Copy link

@akokoshn akokoshn commented Nov 20, 2023

Closes #287. Closes #340.

@akokoshn akokoshn self-assigned this Nov 20, 2023
@akokoshn akokoshn marked this pull request as draft November 20, 2023 07:04
@akokoshn akokoshn force-pushed the 340-lookup-support-multiprover branch 5 times, most recently from 05b865f to f12554d Compare November 24, 2023 10:05
Copy link

@Iluvmagick Iluvmagick left a comment

Choose a reason for hiding this comment

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

Still need to run the tests with proofs.

@akokoshn akokoshn force-pushed the 340-lookup-support-multiprover branch 21 times, most recently from 44e5283 to 48a3061 Compare December 1, 2023 06:52
@martun martun self-requested a review December 1, 2023 09:40
@nkaskov nkaskov force-pushed the 340-lookup-support-multiprover branch from 48a3061 to 07cd01b Compare December 1, 2023 11:20
@nkaskov nkaskov self-requested a review December 1, 2023 11:20
@nkaskov nkaskov changed the title Enable lookup Enable lookup [SyncWith:zkllvm:#340] Dec 1, 2023
@x-mass x-mass changed the title Enable lookup [SyncWith:zkllvm:#340] Enable lookup [SyncWith:zkllvm#340] Dec 1, 2023
@nkaskov nkaskov force-pushed the 340-lookup-support-multiprover branch from 07cd01b to 9ea5f45 Compare December 1, 2023 11:42
@nkaskov nkaskov marked this pull request as ready for review December 1, 2023 11:43
@nil-foundation-cicd-bot
Copy link

Tests will be run in sync with other PRs containing [SyncWith NilFoundation/zkLLVM#340] in title. You can find related PRs as linked with mentioned issue.

@nkaskov nkaskov force-pushed the 340-lookup-support-multiprover branch 2 times, most recently from c99fc12 to bf63304 Compare December 1, 2023 12:39
@akokoshn akokoshn force-pushed the 340-lookup-support-multiprover branch 2 times, most recently from 8a8ada9 to 067c3a7 Compare December 1, 2023 14:29
@akokoshn akokoshn force-pushed the 340-lookup-support-multiprover branch from 067c3a7 to e983c10 Compare December 1, 2023 14:46
@nkaskov nkaskov enabled auto-merge (rebase) December 1, 2023 17:00
@akokoshn
Copy link
Author

akokoshn commented Dec 2, 2023

@Iluvmagick , manually checked proving sha256 in multi provers mode (1th prover has no lookup, 2th - sha256 with lookup)

@nkaskov nkaskov merged commit 6d8005c into master Dec 4, 2023
@nkaskov nkaskov deleted the 340-lookup-support-multiprover branch December 4, 2023 12:07
@nkaskov nkaskov mentioned this pull request Dec 4, 2023
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.

Adding lookup table support Add recursive mode

4 participants