Skip to content
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

x86 optimization for convolution int8 gemm #5874

Merged
merged 36 commits into from
Feb 8, 2025
Merged

Conversation

nihui
Copy link
Member

@nihui nihui commented Jan 15, 2025

  • conv int8 test 0.5 filtered
  • coverage
  • benchmark for avx2 avx512vnni xop

@github-actions github-actions bot added the x86 label Jan 15, 2025
@codecov-commenter
Copy link

codecov-commenter commented Jan 15, 2025

Codecov Report

Attention: Patch coverage is 99.25651% with 2 lines in your changes missing coverage. Please review.

Project coverage is 95.83%. Comparing base (4a70be4) to head (01e3c91).
Report is 1 commits behind head on master.

Files with missing lines Patch % Lines
src/layer/x86/requantize_x86.cpp 90.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #5874      +/-   ##
==========================================
+ Coverage   94.59%   95.83%   +1.23%     
==========================================
  Files         648      762     +114     
  Lines      219000   238772   +19772     
==========================================
+ Hits       207172   228826   +21654     
+ Misses      11828     9946    -1882     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@nihui nihui closed this Jan 15, 2025
@nihui nihui reopened this Jan 15, 2025
@nihui nihui changed the title [WIP] x86 optimization for convolution int8 gemm x86 optimization for convolution int8 gemm Jan 17, 2025
@github-actions github-actions bot added the test label Feb 7, 2025
@nihui nihui merged commit 2389090 into Tencent:master Feb 8, 2025
77 of 80 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants