Skip to content
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.

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

batched logup in opcodes #266

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

ohad-starkware
Copy link
Contributor

@ohad-starkware ohad-starkware commented Dec 19, 2024

This change is Reviewable

@ohad-starkware ohad-starkware mentioned this pull request Dec 19, 2024
Copy link
Contributor Author

ohad-starkware commented Dec 19, 2024

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

@ohad-starkware ohad-starkware mentioned this pull request Dec 19, 2024
@ohad-starkware ohad-starkware force-pushed the ohad/update_infra_names branch from 0acea86 to a969d6d Compare December 19, 2024 09:11
@ohad-starkware ohad-starkware force-pushed the ohad/batched_opcodes_logup branch from 91602fe to 49f4be3 Compare December 19, 2024 09:11
@ohad-starkware ohad-starkware force-pushed the ohad/update_infra_names branch 2 times, most recently from 8f754b4 to 479285b Compare December 19, 2024 09:14
@ohad-starkware ohad-starkware force-pushed the ohad/batched_opcodes_logup branch from 49f4be3 to 4098878 Compare December 19, 2024 09:14
@ohad-starkware ohad-starkware changed the base branch from ohad/update_infra_names to graphite-base/266 December 19, 2024 11:27
@ohad-starkware ohad-starkware force-pushed the ohad/batched_opcodes_logup branch from 4098878 to dc3a595 Compare December 19, 2024 11:27
@ohad-starkware ohad-starkware changed the base branch from graphite-base/266 to main December 19, 2024 11:28
@ohad-starkware ohad-starkware force-pushed the ohad/batched_opcodes_logup branch from dc3a595 to 476c0a4 Compare December 19, 2024 11:28
Copy link
Contributor

@shaharsamocha7 shaharsamocha7 left a comment

Choose a reason for hiding this comment

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

Reviewed 25 of 50 files at r1, 24 of 25 files at r2, all commit messages.
Reviewable status: 49 of 50 files reviewed, all discussions resolved

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