root
/
xz
mirror of https://git.tukaani.org/xz.git
1
0
Fork 0
Code Issues Releases Wiki Activity

liblzma: CRC CLMUL: Omit is_arch_extension_supported() when not needed 

On E2K the function compiles only due to compiler emulation but the
function is never used. It's cleaner to omit the function when it's
not needed even though it's a "static inline" function.

Thanks to Ilya Kurdyukov.

(cherry picked from commit 30a2d5d510)
This commit is contained in:
Lasse Collin 2024-06-16 13:39:37 +03:00
parent 4e4a568f6a
commit ff697eb154
No known key found for this signature in database
GPG Key ID: 38EE757D69184620
1 changed files with 4 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/liblzma/check/crc_x86_clmul.h
View File

@ -385,6 +385,9 @@ crc64_arch_optimized(const uint8_t *buf, size_t size, uint64_t crc)
#endif // BUILDING_CRC64_CLMUL #endif // BUILDING_CRC64_CLMUL
// Even though this is an inline function, compile it only when needed.
// This way it won't appear in E2K builds at all.
#if defined(CRC32_GENERIC) || defined(CRC64_GENERIC)
// Inlining this function duplicates the function body in crc32_resolve() and // Inlining this function duplicates the function body in crc32_resolve() and
// crc64_resolve(), but this is acceptable because this is a tiny function. // crc64_resolve(), but this is acceptable because this is a tiny function.
static inline bool static inline bool
@ -426,3 +429,4 @@ is_arch_extension_supported(void)
// code as is it only reads a variable set at startup but a few bytes // code as is it only reads a variable set at startup but a few bytes
// doesn't matter here. // doesn't matter here.
} }
#endif