d6
/
xz-analysis-mirror
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Revert "liblzma: CLMUL CRC64: Workaround a bug in MSVC (VS2015-2022)." 

This reverts commit 36edc65ab4.

It was reported that it wasn't a good enough fix and MSVC
still produced (different kind of) bad code when building
for 32-bit x86 if optimizations are enabled.

Thanks to Iouri Kharon.
This commit is contained in:
Lasse Collin 2023-01-10 12:47:16 +02:00
parent e81b9fc48c
commit 52902ad695
1 changed files with 0 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/liblzma/check/crc64_fast.c
View File

@ -279,12 +279,6 @@ crc64_clmul(const uint8_t *buf, size_t size, uint64_t crc)
const __m128i initial_crc = _mm_cvtsi64_si128(~crc); const __m128i initial_crc = _mm_cvtsi64_si128(~crc);
#endif #endif
#if defined(_MSC_VER) && !defined(__INTEL_COMPILER) && !defined(__clang__) \
&& defined(_M_IX86)
// Workaround for MSVC when compiling for 32-bit x86:
__asm mov ebx, ebx ; force store ebx: VS2015-2022 destroy it :(
#endif
__m128i v0, v1, v2, v3; __m128i v0, v1, v2, v3;
#ifndef CRC_USE_GENERIC_FOR_SMALL_INPUTS #ifndef CRC_USE_GENERIC_FOR_SMALL_INPUTS