Skip to content

fix: resolve compiler warnings in binary and miscellaneous Index files#4940

Closed
maxwbuckley wants to merge 3 commits intofacebookresearch:mainfrom
maxwbuckley:fix/warnings-index-binary-misc
Closed

fix: resolve compiler warnings in binary and miscellaneous Index files#4940
maxwbuckley wants to merge 3 commits intofacebookresearch:mainfrom
maxwbuckley:fix/warnings-index-binary-misc

Conversation

@maxwbuckley
Copy link
Copy Markdown
Contributor

Summary

  • Fix compiler warnings in Index2Layer, IndexAdditiveQuantizer, IndexAdditiveQuantizerFastScan, IndexBinaryFlat, IndexBinaryFromFloat, IndexBinaryHash, IndexBinaryIVF, IndexFastScan, IndexIDMap, IndexLSH, IndexLattice
  • Fixes include -Wshadow, -Wunused-parameter, and signed/unsigned comparisons

All changes are mechanical. No functional changes.

Part 9/13 of the compiler warnings cleanup (split from #4810 per maintainer request).

Test plan

  • No functional changes — all fixes are mechanical renames and annotations

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 4.6 (1M context) noreply@anthropic.com

@meta-codesync
Copy link
Copy Markdown
Contributor

meta-codesync bot commented Mar 17, 2026

@alibeklfc has imported this pull request. If you are a Meta employee, you can view this in D96996483.

@maxwbuckley maxwbuckley force-pushed the fix/warnings-index-binary-misc branch from ab26f9f to a8190e3 Compare March 18, 2026 10:06
@alibeklfc
Copy link
Copy Markdown
Contributor

Hi, @maxwbuckley. Thank you for your contributions!

Could you please rebase all your PRs on the current version of FAISS and resolve any conflicts?

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@maxwbuckley maxwbuckley force-pushed the fix/warnings-index-binary-misc branch from a8190e3 to b4f71aa Compare March 19, 2026 20:29
@maxwbuckley
Copy link
Copy Markdown
Contributor Author

Done — all PRs have been rebased onto current main and conflicts resolved.

One PR (#4958, part 7 — SIMD/sorting/misc utilities) was closed as its changes are already upstream.

The remaining 11 PRs are now up to date:

maxwbuckley and others added 2 commits March 20, 2026 09:58
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
The local variable qbs was renamed to qbs_ to avoid shadowing the
member variable, but one reference was missed in the
accumulate_loop_qbs call, causing it to use the wrong value.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@meta-codesync
Copy link
Copy Markdown
Contributor

meta-codesync bot commented Mar 20, 2026

@alibeklfc merged this pull request in e5114c2.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants