]> git.ipfire.org Git - thirdparty/xz.git/commitdiff
liblzma: Add a note why crc_always_inline exists for now.
authorLasse Collin <lasse.collin@tukaani.org>
Mon, 30 Oct 2023 16:09:53 +0000 (18:09 +0200)
committerLasse Collin <lasse.collin@tukaani.org>
Mon, 30 Oct 2023 16:44:32 +0000 (18:44 +0200)
Solaris Studio is a possible example (not tested) which
supports the always_inline attribute but might not get
detected by the common.h #ifdefs.

src/liblzma/check/crc_common.h

index 1783b5e76d519f7820675406efb4bb39ee63f0a4..4764eb1735a3c00077ee9feed55505a089e7b1e3 100644 (file)
 // If both __forceinline and inline are used, MSVC will issue a warning.
 // Since MSVC's keyword is a replacement keyword, the lzma_always_inline
 // macro must also contain the inline keyword when its not used in MSVC.
+//
+// NOTE: This doesn't use lzma_always_inline for now as support for it is
+// detected using preprocessor macros which might miss a compiler that
+// does support it. All compilers that support the CLMUL code support
+// the attribute too; if not, we will hopefully get a bug report.
 #ifdef _MSC_VER
 #      define crc_always_inline __forceinline
 #else