Add LoongArch64 support - #1812
Conversation
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: Chris-wh233 The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
Welcome @Chris-wh233! It looks like this is your first PR to zilliztech/knowhere 🎉 |
|
@Chris-wh233 🔍 Important: PR Classification Needed! For efficient project management and a seamless review process, it's essential to classify your PR correctly. Here's how:
For any PR outside the kind/improvement category, ensure you link to the associated issue using the format: “issue: #”. Thanks for your efforts and contribution to the community!. |
|
/kind improvement |
|
@Chris-wh233 Thanks for the contribution, we're taking a look. You may also wish to add the code to the Faiss (https://github.com/facebookresearch/faiss) baseline that this library depends on. |
This PR adds basic support for LoongArch64. The minimum target CPU is a LoongArch64 CPU that supports LSX.
Native build and testing have passed.
Debian forky
GCC/G++ 14
Conan 2.28.1
CPU with LSX and LASX
Cross-build validation
A separate GitHub Actions workflow successfully completed:
GCC 14 LoongArch64 cross-build
Sparse codec source smoke tests
Full Knowhere build and selected unit tests
Workflow run:
https://github.com/Chris-wh233/knowhere/actions/runs/34106005065
Corresponding commit:
Chris-wh233@cbaeffe