-
Notifications
You must be signed in to change notification settings - Fork 129
chore(l1): add rlp encode benches #5413
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
base: main
Are you sure you want to change the base?
Conversation
Lines of code reportTotal lines added: Detailed view |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we should aim at keeping benches in benches subdir. I know a few of them are in other places (and I'm guilty of a few), but keeping crates free from this code would reduce noise in the day to day work and save us from overestimating our line count (assuming we only care about the code that implements functionality, rather than code that measures its performance or tests it).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@mpaulucci @Arkenan wdyt?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't mind either way, but we should be consitent. If we go for the top level benches lets create a ticket to move all benches there
RLP Bench ResultsBench output |
Adds the encoding benches for rlp
Part of #5411