Skip to content

perf(processor): drop dead clk arg from u32 range-check#3135

Merged
Nashtare merged 2 commits into
nextfrom
robin/dead
May 11, 2026
Merged

perf(processor): drop dead clk arg from u32 range-check#3135
Nashtare merged 2 commits into
nextfrom
robin/dead

Conversation

@Nashtare
Copy link
Copy Markdown
Contributor

Describe your changes

Since #3119 dropped cycle_lookups, the clk parameter in Tracer::record_u32_range_checks is basically dead so can be removed to prune 4 bytes per insert.

Checklist before requesting a review

  • Repo forked and branch created from next according to naming convention.
  • Commit messages and codestyle follow conventions.
  • Commits are signed.
  • Relevant issues are linked in the PR description.
  • Tests added for new functionality.
  • Documentation/comments updated according to changes.
  • Updated `CHANGELOG.md'

@Nashtare Nashtare requested review from huitseeker and plafer May 11, 2026 06:54
@Nashtare Nashtare self-assigned this May 11, 2026
Copy link
Copy Markdown
Contributor

@adr1anh adr1anh left a comment

Choose a reason for hiding this comment

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

Looks good!

@Nashtare Nashtare merged commit 9ce6e9e into next May 11, 2026
23 checks passed
@Nashtare Nashtare deleted the robin/dead branch May 11, 2026 11:02
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