From 0fb56d94b7f409cd47f970b6e721deb1a837d8aa Mon Sep 17 00:00:00 2001 From: Richard Barnes Date: Tue, 12 Nov 2024 10:34:57 -0800 Subject: [PATCH] Remove unused-variable in dumbo/backup/dumbo/service/tests/ChainReplicatorTests.cpp +3 (#4024) Summary: Pull Request resolved: https://github.com/facebookresearch/faiss/pull/4024 LLVM-15 has a warning `-Wunused-variable` which we treat as an error because it's so often diagnostic of a code issue. Unused variables can compromise readability or, worse, performance. This diff either (a) removes an unused variable and, possibly, it's associated code or (b) qualifies the variable with `[[maybe_unused]]`. #buildsonlynotests - Builds are sufficient - If you approve of this diff, please use the "Accept & Ship" button :-) Differential Revision: D65755277 fbshipit-source-id: 13a2ad06375fd84e5e7afd69488e7fa36b658f20 --- faiss/utils/distances_simd.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/faiss/utils/distances_simd.cpp b/faiss/utils/distances_simd.cpp index ecf7e74146..627300a5bc 100644 --- a/faiss/utils/distances_simd.cpp +++ b/faiss/utils/distances_simd.cpp @@ -3759,7 +3759,7 @@ void fvec_add(size_t d, const float* a, float b, float* c) { size_t i; simd8float32 bv(b); for (i = 0; i + 7 < d; i += 8) { - simd8float32 ci, ai, bi; + simd8float32 ci, ai; ai.loadu(a + i); ci = ai + bv; ci.storeu(c + i);