Utils and libraries: fix static_assert for macro HAVE_THREAD_LOCAL if define DEBUG_LOCKCONTENTION
Utils and libraries: fix static_assert for macro HAVE_THREAD_LOCAL #15970
pull orientye wants to merge 1 commits into bitcoin:master from orientye:master changing 1 files +4 −0-
orientye commented at 6:15 AM on May 7, 2019: contributor
-
41ab2a8924
fix static_assert for macro HAVE_THREAD_LOCAL
fix static_assert for macro HAVE_THREAD_LOCAL while define DEBUG_LOCKCONTENTION
- orientye renamed this:
fix static_assert for macro HAVE_THREAD_LOCAL
Utils and libraries: fix static_assert for macro HAVE_THREAD_LOCAL
on May 7, 2019 - fanquake added the label Utils/log/libs on May 7, 2019
-
orientye commented at 7:22 AM on May 10, 2019: contributor
The clang compiler give static_assert err if define macro DEBUG_LOCKCONTENTION on mac osx.(HAVE_THREAD_LOCAL was defined in confiure.ac, and thread_local is supported by the os)
here is the src: #ifdef DEBUG_LOCKCONTENTION #if !defined(HAVE_THREAD_LOCAL) static_assert(false, "thread_local is not supported"); #endif because it lacks include <config/bitcoin-config.h> in sync.cpp for HAVE_THREAD_LOCAL macro.
This PR has fixed this bug.
-
laanwj commented at 6:17 PM on May 16, 2019: member
utACK 41ab2a892456f4f3e27128a7f9237bc45cc288dd
- laanwj merged this on May 16, 2019
- laanwj closed this on May 16, 2019
- laanwj referenced this in commit 8f2f17f79a on May 16, 2019
- sidhujag referenced this in commit 206e65aa51 on May 18, 2019
- luke-jr referenced this in commit 62458b5f4b on Sep 3, 2019
- PastaPastaPasta referenced this in commit 4d46f22b7f on Jun 27, 2021
- PastaPastaPasta referenced this in commit 088221f0a0 on Jun 28, 2021
- PastaPastaPasta referenced this in commit c0176bcb05 on Jun 29, 2021
- PastaPastaPasta referenced this in commit 01661d453c on Sep 11, 2021
- PastaPastaPasta referenced this in commit 5484390217 on Sep 11, 2021
- PastaPastaPasta referenced this in commit 661222fb57 on Sep 12, 2021
- DrahtBot locked this on Dec 16, 2021
Labels