Skip to content

Commit 2ff9110

Browse files
authored
Remove SIMDJSON_DO_NOT_USE_THREADS_NO_MATTER_WHAT (simdjson#1131)
1 parent a954d50 commit 2ff9110

File tree

2 files changed

+0
-17
lines changed

2 files changed

+0
-17
lines changed

cmake/simdjson-flags.cmake

Lines changed: 0 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -133,17 +133,6 @@ if(SIMDJSON_ENABLE_THREADS)
133133
target_compile_definitions(simdjson-flags INTERFACE SIMDJSON_THREADS_ENABLED=1) # This will be set in the code automatically.
134134
endif()
135135

136-
# Some users compile simdjson with thread support but still do not want simdjson to use threads.
137-
#
138-
# Important : Expect this option to disappear in the future.
139-
#
140-
option(SIMDJSON_DO_NOT_USE_THREADS_NO_MATTER_WHAT "Whether we enabled thread support or not (SIMDJSON_ENABLE_THREADS), do not use threads.\
141-
This option does nothing when thread support is not enabled. We reserve the right to remove this option in a future release in\
142-
favor of a runtime approach." OFF)
143-
if(SIMDJSON_DO_NOT_USE_THREADS_NO_MATTER_WHAT)
144-
target_compile_definitions(simdjson-flags INTERFACE SIMDJSON_DO_NOT_USE_THREADS_NO_MATTER_WHAT=1)
145-
endif()
146-
147136
if(SIMDJSON_USE_LIBCPP)
148137
target_link_libraries(simdjson-flags INTERFACE -stdlib=libc++ -lc++abi)
149138
# instead of the above line, we could have used

include/simdjson/portability.h

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -148,12 +148,6 @@ use a 64-bit target such as x64 or 64-bit ARM.")
148148
#endif
149149
150150
151-
#if SIMDJSON_DO_NOT_USE_THREADS_NO_MATTER_WHAT
152-
// No matter what happened, we undefine SIMDJSON_THREADS_ENABLED and so disable threads.
153-
#undef SIMDJSON_THREADS_ENABLED
154-
#endif
155-
156-
157151
#if defined(__clang__)
158152
#define NO_SANITIZE_UNDEFINED __attribute__((no_sanitize("undefined")))
159153
#elif defined(__GNUC__)

0 commit comments

Comments
 (0)