Skip to content

speed up fast_bitset by use simd operators #764

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

Merged
merged 1 commit into from
May 29, 2025
Merged

Conversation

LHT129
Copy link
Collaborator

@LHT129 LHT129 commented May 26, 2025

No description provided.

@LHT129 LHT129 requested a review from inabao May 26, 2025 08:48
@LHT129 LHT129 self-assigned this May 26, 2025
@LHT129 LHT129 requested a review from ShawnShawnYou as a code owner May 26, 2025 08:48
@LHT129 LHT129 added kind/improvement Code improvements (variable/function renaming, refactoring, etc. ) version/0.15 labels May 26, 2025
@pull-request-size pull-request-size bot added size/M and removed size/S labels May 26, 2025
@LHT129 LHT129 requested review from wxyucs and jiaweizone as code owners May 26, 2025 09:44
@pull-request-size pull-request-size bot added size/L and removed size/M labels May 26, 2025
Copy link

mergify bot commented May 26, 2025

⚠️ The sha of the head commit of this PR conflicts with #761. Mergify cannot evaluate rules on this PR. ⚠️

Copy link

codecov bot commented May 26, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

❌ Your project status has failed because the head coverage (88.78%) is below the target coverage (90.00%). You can increase the head coverage or adjust the target coverage.

@@            Coverage Diff             @@
##             main     #764      +/-   ##
==========================================
- Coverage   90.56%   88.78%   -1.78%     
==========================================
  Files         195      198       +3     
  Lines       12071    12405     +334     
==========================================
+ Hits        10932    11014      +82     
- Misses       1139     1391     +252     
Flag Coverage Δ
cpp 88.78% <100.00%> (-1.78%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Components Coverage Δ
common 92.51% <ø> (ø)
datacell 91.66% <ø> (+0.03%) ⬆️
index 88.74% <ø> (ø)
simd 100.00% <ø> (ø)

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 200adc1...92a82aa. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Collaborator

@wxyucs wxyucs left a comment

Choose a reason for hiding this comment

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

lgtm

@LHT129 LHT129 merged commit 83450ff into antgroup:main May 29, 2025
28 of 30 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/improvement Code improvements (variable/function renaming, refactoring, etc. ) size/M version/0.15
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants