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

Update gcc to 12.4 to support avx512_fp16 in k-NN faiss lib and node20 on arm64 #5241

Merged
merged 6 commits into from
Jan 22, 2025

Conversation

peterzhuamazon
Copy link
Member

@peterzhuamazon peterzhuamazon commented Jan 15, 2025

Description

Update gcc to 12.4 to support avx512_fp16 in k-NN faiss lib and node20 on arm64

Issues Resolved

Closes #5226
Also follow up: #5178
Part of #5248

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Signed-off-by: Peter Zhu <[email protected]>
Copy link

codecov bot commented Jan 15, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 92.12%. Comparing base (13225f2) to head (e676b85).
Report is 18 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #5241      +/-   ##
==========================================
- Coverage   92.35%   92.12%   -0.24%     
==========================================
  Files         197      202       +5     
  Lines        6814     7033     +219     
==========================================
+ Hits         6293     6479     +186     
- Misses        521      554      +33     

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

Signed-off-by: Peter Zhu <[email protected]>
@rishabh6788 rishabh6788 marked this pull request as ready for review January 22, 2025 22:01
@peterzhuamazon peterzhuamazon changed the title Update gcc to 12.4 to support avx512_fp16 in k-NN faiss lib Update gcc to 12.4 to support avx512_fp16 in k-NN faiss lib and node20 on arm64 Jan 22, 2025
Signed-off-by: Peter Zhu <[email protected]>
@peterzhuamazon peterzhuamazon merged commit f7b47ec into opensearch-project:main Jan 22, 2025
14 checks passed
@peterzhuamazon peterzhuamazon deleted the al2-gcc13-knn branch January 22, 2025 22:15
@peterzhuamazon
Copy link
Member Author

@peterzhuamazon
Copy link
Member Author

Building took way too long over 5 hours, would need to reduce the times on this.

@peterzhuamazon
Copy link
Member Author

#5252

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

Bump up GCC version to 12.x or later to support SIMD AVX512 fp16 instructions
2 participants