Commit 63edc32
Fix AVX512 build targets in CMakeLists.txt (facebookresearch#3214)
Summary:
When `FAISS_OPT_LEVEL=avx512`, the current default target omits the `avx2` build, which conflicts with the install targets that specify both `avx2` and `avx512` builds. This PR fixes the default build targets for AVX512.
Install targets:
https://github.com/facebookresearch/faiss/blob/e19de27d72b47bb8782bc173725c45c46c9782eb/faiss/CMakeLists.txt#L332
With this PR, cmake can build and install without any error.
```bash
cmake -B build -DFAISS_OPT_LEVEL=avx512 .
cmake --build build --config Release -j
cmake --install build
```
Pull Request resolved: facebookresearch#3214
Reviewed By: mlomeli1
Differential Revision: D52996716
Pulled By: algoriddle
fbshipit-source-id: b8a46eee6cc15c2043a1a74c5e15d7a606e94acc1 parent 99c0e2b commit 63edc32
1 file changed
+1
-1
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
230 | 230 | | |
231 | 231 | | |
232 | 232 | | |
233 | | - | |
| 233 | + | |
234 | 234 | | |
235 | 235 | | |
236 | 236 | | |
| |||
0 commit comments