Skip to content

Commit 5ccf546

Browse files
authored
[llvm][cmake] Performing expensive checks requires enabling assert. (llvm#80821)
LLVM will intercept errors using assert() when LLVM_ENABLE_EXPENSIVE_CHECKS is ON. So an explicit check is added. --------- Co-authored-by: Hua Tian <[email protected]>
1 parent 19e518d commit 5ccf546

File tree

1 file changed

+5
-0
lines changed

1 file changed

+5
-0
lines changed

llvm/cmake/modules/HandleLLVMOptions.cmake

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -128,6 +128,11 @@ if("${LLVM_DEFAULT_TARGET_TRIPLE}" MATCHES "^amdgcn" OR
128128
endif()
129129

130130
if(LLVM_ENABLE_EXPENSIVE_CHECKS)
131+
# When LLVM_ENABLE_EXPENSIVE_CHECKS is ON, LLVM will intercept errors
132+
# using assert(). An explicit check is performed here.
133+
if (NOT LLVM_ENABLE_ASSERTIONS)
134+
message(FATAL_ERROR "LLVM_ENABLE_EXPENSIVE_CHECKS requires LLVM_ENABLE_ASSERTIONS \"ON\".")
135+
endif()
131136
add_compile_definitions(EXPENSIVE_CHECKS)
132137

133138
# In some libstdc++ versions, std::min_element is not constexpr when

0 commit comments

Comments
 (0)