Skip to content

move distances implementation into SIMD specific compile units#4906

Closed
mdouze wants to merge 1 commit into
facebookresearch:mainfrom
mdouze:export-D95936389
Closed

move distances implementation into SIMD specific compile units#4906
mdouze wants to merge 1 commit into
facebookresearch:mainfrom
mdouze:export-D95936389

Conversation

@mdouze
Copy link
Copy Markdown
Contributor

@mdouze mdouze commented Mar 11, 2026

Summary:
The main objective is to move the IVFFlat scanning code inside SIMD compile units, so that the loop over codes runs inside SIMD code.

Unfortunately, I could not instant
iate all the implementations with with_MetricType, hence the macro to define them.

Reviewed By: algoriddle

Differential Revision: D95936389

@meta-cla meta-cla Bot added the CLA Signed label Mar 11, 2026
@meta-codesync
Copy link
Copy Markdown
Contributor

meta-codesync Bot commented Mar 11, 2026

@mdouze has exported this pull request. If you are a Meta employee, you can view the originating Diff in D95936389.

mdouze added a commit to mdouze/faiss that referenced this pull request Mar 11, 2026
…ookresearch#4906)

Summary:

The main objective is to move the IVFFlat scanning code inside SIMD compile units, so that the loop over codes runs inside SIMD code.

Unfortunately, I could not instant
iate all the implementations with with_MetricType, hence the macro to define them.

Reviewed By: algoriddle

Differential Revision: D95936389
@mdouze mdouze force-pushed the export-D95936389 branch from e6f7bb5 to f9ce00d Compare March 11, 2026 21:21
…ookresearch#4906)

Summary:

The main objective is to move the IVFFlat scanning code inside SIMD compile units, so that the loop over codes runs inside SIMD code.

Unfortunately, I could not instant
iate all the implementations with with_MetricType, hence the macro to define them.

Reviewed By: algoriddle

Differential Revision: D95936389
@meta-codesync
Copy link
Copy Markdown
Contributor

meta-codesync Bot commented Mar 12, 2026

This pull request has been merged in d8c1a97.

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