Skip to content

Optimize rearrange_tracks kernel #1144

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
Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Optimize rearrange_tracks kernel #1144

wants to merge 1 commit into from

Conversation

@beomki-yeo
Copy link
Contributor

@beomki-yeo beomki-yeo commented Aug 30, 2025

Another PR assisted by AI. The change is nothing drastic but clearing up the codes and reducing the number of atomic operations.

Change summary:

  • Do local accumulation per lane; emit at most one atomicAdd if non-zero.
  • Merge offset search + left-count into one pass; cache lane/group/stride etc.
  • Make the valid-slot binary search leader-only to cut redundant work.
  • Fix int delta = delta = ... and hoist reference loads into const locals.

@beomki-yeo beomki-yeo added improvement Improve an existing feature AI assistance Assisted by AI labels Aug 30, 2025
@sonarqubecloud
Copy link

sonarqubecloud bot commented Aug 30, 2025

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@beomki-yeo beomki-yeo marked this pull request as draft September 22, 2025 23:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

AI assistance Assisted by AI improvement Improve an existing feature

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@beomki-yeo