Skip to content

Commit

Permalink
Merge commit 6b761503ba073688e3ebeaa60545633dad83d1ba from Faiss mast…
Browse files Browse the repository at this point in the history
…er branch

Commit: 6b761503ba073688e3ebeaa60545633dad83d1ba
Parents: 6c89c8bd4e239141f9a5fad224347b80114ea585
Author: chasingegg <[email protected]>
Committer: Facebook GitHub Bot <[email protected]>
Date: Thu Nov 02 2023 13:03:52 GMT-0400 (Eastern Daylight Time)

Remove confusing comments in partitioning.cpp (#3104)

Summary:
Fix facebookresearch/faiss#3095

Pull Request resolved: facebookresearch/faiss#3104

Reviewed By: mlomeli1

Differential Revision: D50595865

Pulled By: mdouze

fbshipit-source-id: f9107bda114a77d6e1f0da32c3451b7182d32e60
Signed-off-by: Alexandr Guzhva <[email protected]>
  • Loading branch information
alexanderguzhva committed Nov 27, 2023
1 parent a65e344 commit bb71c78
Showing 1 changed file with 0 additions and 2 deletions.
2 changes: 0 additions & 2 deletions thirdparty/faiss/faiss/utils/partitioning.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -754,8 +754,6 @@ typename C::T partition_fuzzy(
size_t q_min,
size_t q_max,
size_t* q_out) {
// the code below compiles and runs without AVX2 but it's slower than
// the scalar implementation
#ifdef __AVX2__
constexpr bool is_uint16 = std::is_same<typename C::T, uint16_t>::value;
if (is_uint16 && is_aligned_pointer(vals)) {
Expand Down

0 comments on commit bb71c78

Please sign in to comment.