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

Fixed _mm256_set_m128 is only availble on gcc8+. issue#5072 #5075

Merged
merged 23 commits into from
Oct 10, 2023

Conversation

whyb
Copy link
Contributor

@whyb whyb commented Oct 10, 2023

Fixed _mm256_set_m128 is only availble on gcc8+. issue #5072

@tencent-adm
Copy link
Member

CLA assistant check
Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

✅ nihui
❌ whyb
You have signed the CLA already but the status is still pending? Let us recheck it.

@codecov-commenter
Copy link

codecov-commenter commented Oct 10, 2023

Codecov Report

Merging #5075 (643816e) into master (75ad1cc) will decrease coverage by 0.01%.
Report is 2 commits behind head on master.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #5075      +/-   ##
==========================================
- Coverage   94.73%   94.73%   -0.01%     
==========================================
  Files         767      767              
  Lines      230160   230161       +1     
==========================================
  Hits       218049   218049              
- Misses      12111    12112       +1     
Files Coverage Δ
src/layer/x86/shufflechannel_x86.cpp 97.51% <100.00%> (+<0.01%) ⬆️

... and 1 file with indirect coverage changes

@nihui nihui merged commit b4f8fa6 into Tencent:master Oct 10, 2023
56 of 57 checks passed
@nihui
Copy link
Member

nihui commented Oct 10, 2023

Thanks for your contribution !

@whyb whyb deleted the issue_5072 branch October 10, 2023 08:08
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.

4 participants