Skip to content

Commit

Permalink
Remove unused-variable in dumbo/backup/dumbo/service/tests/ChainRepli…
Browse files Browse the repository at this point in the history
…catorTests.cpp +3 (facebookresearch#4024)

Summary:
Pull Request resolved: facebookresearch#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
  • Loading branch information
r-barnes authored and facebook-github-bot committed Nov 12, 2024
1 parent adb1884 commit 0fb56d9
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion faiss/utils/distances_simd.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -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);
Expand Down

0 comments on commit 0fb56d9

Please sign in to comment.