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

Avoid use of NDEBUG in source files 

Posix NDEBUG causes our test script to fail the Posix Assert test
pull/757/head
Jeffrey Walton 2018-12-03 04:18:41 -05:00
parent 774454ba7c
commit 318d53f6f9
No known key found for this signature in database
GPG Key ID: B36AB348921B1838
2 changed files with 10 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
chacha_avx.cpp
View File

@ -42,10 +42,12 @@ extern const char CHACHA_AVX_FNAME[] = __FILE__;
// https://github.com/weidai11/cryptopp/issues/735. The
// 649 issue affects AES but it is the same here. The 735
// issue is ChaCha AVX2 cut-in where it surfaced again.
#if (_MSC_VER >= 1910) && defined(NDEBUG)
#if (_MSC_VER >= 1910)
# ifndef CRYPTOPP_DEBUG
# pragma optimize("", off)
# pragma optimize("ts", on)
# endif
#endif
ANONYMOUS_NAMESPACE_BEGIN

4
rijndael.cpp
View File

@ -91,10 +91,12 @@ being unloaded from L1 cache, until that round is finished.
// VS2017 and global optimization bug. TODO, figure out when
// we can re-enable full optimizations for VS2017. Also see
// https://github.com/weidai11/cryptopp/issues/649
#if (_MSC_VER >= 1910) && defined(NDEBUG)
#if (_MSC_VER >= 1910)
# ifndef CRYPTOPP_DEBUG
# pragma optimize("", off)
# pragma optimize("ts", on)
# endif
#endif
NAMESPACE_BEGIN(CryptoPP)