Skip to content

Commit

Permalink
avx512 compilation option (#3798)
Browse files Browse the repository at this point in the history
Summary:
Pull Request resolved: #3798

Alexander left a comment on the previous PR: #3785 (comment). The contract for the function seems to be that it will only append a single compilation option, not a list of options. Fixing it to comply with the contract.

Reviewed By: asadoughi, ramilbakhshyiev

Differential Revision: D61803839

fbshipit-source-id: 948a3d636f6dd6b5c4f975d236c19923af2bbd18
  • Loading branch information
mengdilin authored and facebook-github-bot committed Aug 26, 2024
1 parent 4ca6734 commit 3614cc7
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions faiss/utils/utils.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -115,11 +115,10 @@ std::string get_compile_options() {
options += "OPTIMIZE ";
#endif

#ifdef __AVX2__
options += "AVX2 ";
#ifdef __AVX512F__
options += "AVX512 ";
#endif
#elif defined(__AVX2__)
options += "AVX2 ";
#elif defined(__ARM_FEATURE_SVE)
options += "SVE NEON ";
#elif defined(__aarch64__)
Expand Down

0 comments on commit 3614cc7

Please sign in to comment.