arne
/
cryptopp
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Disable CLMUL again on SunStudio (GH# 188, GH #224) 

We got reports that x86_64 was producing incorrect results. Also, the problem persisted in i386 builds. I don't think we can work around this issue. Oracle must fix it.
pull/668/merge
Jeffrey Walton 2018-07-16 09:39:01 -04:00
parent d1e646a589
commit 9d954efcde
No known key found for this signature in database
GPG Key ID: B36AB348921B1838
2 changed files with 10 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
config.h
View File

@ -585,6 +585,12 @@ NAMESPACE_END
# endif
#endif
// Fixup for SunCC 12.2-12.6. Compiler crash on GCM_Reduce_CLMUL and friends.
// http://github.com/weidai11/cryptopp/issues/226
#if defined(__SUNPRO_CC) && (__SUNPRO_CC <= 0x5150)
# undef CRYPTOPP_CLMUL_AVAILABLE
#endif
#endif // X86, X32, X64
// ***************** ARM CPU features ********************

7
gcm-simd.cpp
View File

@ -10,9 +10,10 @@
#include "config.h"
#include "misc.h"
// Clang 3.3 integrated assembler crash on Linux. Other versions produce incorrect results.
// Clang has never handled Intel ASM very well. I wish LLVM would fix it.
#if defined(__clang__)
// Clang 3.3 integrated assembler crash on Linux. Other versions
// produce incorrect results. Clang has never handled Intel ASM
// very well. I wish LLVM would fix it.
#if defined(CRYPTOPP_DISABLE_INTEL_ASM)
# undef CRYPTOPP_X86_ASM_AVAILABLE
# undef CRYPTOPP_X32_ASM_AVAILABLE
# undef CRYPTOPP_X64_ASM_AVAILABLE