]> git.ipfire.org Git - thirdparty/xz.git/commitdiff
liblzma: CRC CLMUL: Omit is_arch_extension_supported() when not needed 127/head
authorLasse Collin <lasse.collin@tukaani.org>
Sun, 16 Jun 2024 10:39:37 +0000 (13:39 +0300)
committerLasse Collin <lasse.collin@tukaani.org>
Mon, 17 Jun 2024 12:00:55 +0000 (15:00 +0300)
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.

src/liblzma/check/crc_x86_clmul.h

index 702cc20212e777ea7e5d8702be1d753a974e3c88..9264765446a00e0eb8d5b02c397e25013c903a15 100644 (file)
@@ -335,6 +335,9 @@ crc64_arch_optimized(const uint8_t *buf, size_t size, uint64_t crc)
 }
 
 
+// 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
 // crc64_resolve(), but this is acceptable because this is a tiny function.
 static inline bool
@@ -376,3 +379,4 @@ is_arch_extension_supported(void)
        // code as is it only reads a variable set at startup but a few bytes
        // doesn't matter here.
 }
+#endif