Skip to content

Ignore dead_code warnings on scalar bit transpose code#7408

Merged
robert3005 merged 1 commit intodevelopfrom
rk/nowarns
Apr 13, 2026
Merged

Ignore dead_code warnings on scalar bit transpose code#7408
robert3005 merged 1 commit intodevelopfrom
rk/nowarns

Conversation

@robert3005
Copy link
Copy Markdown
Contributor

Likely we shouldn't use expect here but we also don't test on architectures
where this wouldn't be dead code

Signed-off-by: Robert Kruszewski github@robertk.io

Signed-off-by: Robert Kruszewski <github@robertk.io>
@robert3005 robert3005 added the changelog/skip Do not list PR in the changelog label Apr 13, 2026
@robert3005 robert3005 enabled auto-merge (squash) April 13, 2026 12:34
/// This version uses 64-bit gather + parallel bit operations instead of
/// extracting bits one by one. Typically 5-10x faster than the basic scalar version.
#[inline(never)]
#[allow(dead_code)]
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

In practice looks like a if x86

@robert3005 robert3005 merged commit 75b1062 into develop Apr 13, 2026
66 of 68 checks passed
@robert3005 robert3005 deleted the rk/nowarns branch April 13, 2026 12:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog/skip Do not list PR in the changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants