diff options
| author | Lasse Collin <lasse.collin@tukaani.org> | 2023-10-30 18:09:53 +0200 |
|---|---|---|
| committer | Lasse Collin <lasse.collin@tukaani.org> | 2023-10-30 18:44:32 +0200 |
| commit | 8c36ab79cbf23104ce7a3d533d5ac98cd492e57c (patch) | |
| tree | ec87e8f4916f130edf9939e761512395eb694391 | |
| parent | liblzma: Use lzma_always_inline in memcmplen.h. (diff) | |
| download | xz-8c36ab79cbf23104ce7a3d533d5ac98cd492e57c.tar.xz | |
liblzma: Add a note why crc_always_inline exists for now.
Solaris Studio is a possible example (not tested) which
supports the always_inline attribute but might not get
detected by the common.h #ifdefs.
| -rw-r--r-- | src/liblzma/check/crc_common.h | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/src/liblzma/check/crc_common.h b/src/liblzma/check/crc_common.h index 1783b5e7..4764eb17 100644 --- a/src/liblzma/check/crc_common.h +++ b/src/liblzma/check/crc_common.h @@ -48,6 +48,11 @@ // 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 |