Skip to content

Commit

Permalink
Merge pull request #4230 from facebook/variedOutput
Browse files Browse the repository at this point in the history
Do not vary row matchfinder selection based on availability of SSE2/Neon
  • Loading branch information
Victor-C-Zhang authored Jan 6, 2025
2 parents 1548bfc + d88651e commit 59c2e33
Showing 1 changed file with 1 addition and 10 deletions.
11 changes: 1 addition & 10 deletions lib/compress/zstd_compress.c
Original file line number Diff line number Diff line change
Expand Up @@ -237,19 +237,10 @@ static int ZSTD_rowMatchFinderUsed(const ZSTD_strategy strategy, const ZSTD_Para
/* Returns row matchfinder usage given an initial mode and cParams */
static ZSTD_ParamSwitch_e ZSTD_resolveRowMatchFinderMode(ZSTD_ParamSwitch_e mode,
const ZSTD_compressionParameters* const cParams) {
#if defined(ZSTD_ARCH_X86_SSE2) || defined(ZSTD_ARCH_ARM_NEON)
int const kHasSIMD128 = 1;
#else
int const kHasSIMD128 = 0;
#endif
if (mode != ZSTD_ps_auto) return mode; /* if requested enabled, but no SIMD, we still will use row matchfinder */
mode = ZSTD_ps_disable;
if (!ZSTD_rowMatchFinderSupported(cParams->strategy)) return mode;
if (kHasSIMD128) {
if (cParams->windowLog > 14) mode = ZSTD_ps_enable;
} else {
if (cParams->windowLog > 17) mode = ZSTD_ps_enable;
}
if (cParams->windowLog > 14) mode = ZSTD_ps_enable;
return mode;
}

Expand Down

0 comments on commit 59c2e33

Please sign in to comment.